-
Notifications
You must be signed in to change notification settings - Fork 33
Fix device login URL being printed twice #1255
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix device login URL being printed twice #1255
Conversation
d01de49 to
4755c2c
Compare
|
What was the previous output? I feel the fix belongs to the broker though. |
|
That is the previous output (main branch) on my local machine Do you have any specific suggestions or directions on how you'd like to see the fix implemented in the broker instead? |
4755c2c to
a3d2ba6
Compare
|
Thanks for the insight. I've simplifed the PR by removing the device login URL from the label at the broker (because the content already contains the Here is the current output |
340a5ca to
5608ca4
Compare
|
Note that this may require adjusting the e2e tests. |
5608ca4 to
c7631d2
Compare
|
Fixed the failing e2e broker tests and rebased with main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for the contribution!
The change looks good to me. There was just some confusion regarding the "e2e tests". @3v1n0 was referring to the tests in e2e-tests/ but they don't use the string that was changed, so we don't have to update them.
...internal/broker/testdata/golden/TestSelectAuthenticationMode/Successfully_select_device_auth
Outdated
Show resolved
Hide resolved
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1255 +/- ##
==========================================
- Coverage 86.04% 85.26% -0.78%
==========================================
Files 99 119 +20
Lines 6685 7669 +984
Branches 111 111
==========================================
+ Hits 5752 6539 +787
- Misses 877 1074 +197
Partials 56 56 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
c7631d2 to
665fe30
Compare
665fe30 to
db02100
Compare
db02100 to
f6824e5
Compare
adombeck
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks again for the contribution and for addressing all the suggestions!
Closes #682
Current output for SSH login attempt after the fix