feat: Add host CLI option as alternative to config#1395
feat: Add host CLI option as alternative to config#1395
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1395 +/- ##
==========================================
+ Coverage 95.19% 95.22% +0.03%
==========================================
Files 43 43
Lines 3119 3122 +3
==========================================
+ Hits 2969 2973 +4
+ Misses 150 149 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
c3e9908 to
136df8d
Compare
3fde52e to
2d156a3
Compare
docs/reference/openapi.yaml
Outdated
| $ref: '#/components/schemas/BasicAuthentication' | ||
| description: Auth information for communicating with STOMP broker, if required | ||
| title: Auth | ||
| enabled: |
There was a problem hiding this comment.
If we're freely giving the auth config away, is there much point having it?
There was a problem hiding this comment.
No, maybe we should discuss this now. The level of security going on at various points here is getting tenuous. Even if we decide not to do it in this PR (likely), we should have a plan on what we're going to do about it.
2cbe490 to
2533134
Compare
2533134 to
67ee7cb
Compare
67ee7cb to
3d17702
Compare
abbiemery
left a comment
There was a problem hiding this comment.
When trying to run scans against tiled 0.2.4 (current state of main) with this I am getting an error from the tiled writer:
TypeError: DataSource.__init__() got an unexpected keyword argument 'properties'
This may be just down to it needing rebasing but does raise the question of if we are clear somewhere about the versions of tiled we support.
Works against 0.2.3. Can login/out, get devices and run plans etc. Happy to approve pending a credential discussion.
docs/reference/openapi.yaml
Outdated
| $ref: '#/components/schemas/BasicAuthentication' | ||
| description: Auth information for communicating with STOMP broker, if required | ||
| title: Auth | ||
| enabled: |
There was a problem hiding this comment.
No, maybe we should discuss this now. The level of security going on at various points here is getting tenuous. Even if we decide not to do it in this PR (likely), we should have a plan on what we're going to do about it.
|
Keep the new CLI option to set the host but remove the stomp config stuff and replace with websocket work |
3d17702 to
8d41f65
Compare
8d41f65 to
b0f3611
Compare
No description provided.