Skip to content

Conversation

@nooreldeensalah
Copy link
Contributor

@nooreldeensalah nooreldeensalah commented Feb 10, 2026

Closes #682

Current output for SSH login attempt after the fix

== Device Authentication ==
Access the link below and use the provided login code
https://www.google.com/device
        FNC-YVC-MMQX

@nooreldeensalah nooreldeensalah force-pushed the fix-device-login-url-printed-twice branch from d01de49 to 4755c2c Compare February 10, 2026 05:03
@3v1n0
Copy link
Contributor

3v1n0 commented Feb 10, 2026

What was the previous output?

I feel the fix belongs to the broker though.

@nooreldeensalah
Copy link
Contributor Author

nooreldeensalah commented Feb 10, 2026

That is the previous output (main branch) on my local machine

== Device Authentication ==
Access "https://www.google.com/device" and use the provided login code
https://www.google.com/device
        BLY-FHF-KWXW

Do you have any specific suggestions or directions on how you'd like to see the fix implemented in the broker instead?

@nooreldeensalah nooreldeensalah force-pushed the fix-device-login-url-printed-twice branch from 4755c2c to a3d2ba6 Compare February 10, 2026 12:52
@nooreldeensalah
Copy link
Contributor Author

nooreldeensalah commented Feb 10, 2026

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 VerificationURI )

Here is the current output

== Device Authentication ==
Access the link below and use the provided login code
https://www.google.com/device
        FNC-YVC-MMQX

@nooreldeensalah nooreldeensalah force-pushed the fix-device-login-url-printed-twice branch 2 times, most recently from 340a5ca to 5608ca4 Compare February 10, 2026 14:14
@3v1n0
Copy link
Contributor

3v1n0 commented Feb 10, 2026

Note that this may require adjusting the e2e tests.

@nooreldeensalah nooreldeensalah force-pushed the fix-device-login-url-printed-twice branch from 5608ca4 to c7631d2 Compare February 10, 2026 14:45
@nooreldeensalah
Copy link
Contributor Author

Fixed the failing e2e broker tests and rebased with main

@nooreldeensalah nooreldeensalah changed the title fix: ssh login: device login URL is printed twice Fix device login URL being printed twice Feb 10, 2026
Copy link
Contributor

@adombeck adombeck left a 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.

@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.26%. Comparing base (f40fcbe) to head (f6824e5).
⚠️ Report is 6 commits behind head on main.

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.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nooreldeensalah nooreldeensalah force-pushed the fix-device-login-url-printed-twice branch from c7631d2 to 665fe30 Compare February 11, 2026 19:29
@nooreldeensalah nooreldeensalah force-pushed the fix-device-login-url-printed-twice branch from db02100 to f6824e5 Compare February 12, 2026 11:56
Copy link
Contributor

@adombeck adombeck left a 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!

@adombeck adombeck merged commit 1b9ea6b into canonical:main Feb 12, 2026
19 of 29 checks passed
@nooreldeensalah nooreldeensalah deleted the fix-device-login-url-printed-twice branch February 12, 2026 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ssh login: device login URL is printed twice

4 participants