Skip to content
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

test: Don't explicitly go to x0.cockpit.lan in TestIPA #21064

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

mvollmer
Copy link
Member

@mvollmer mvollmer commented Oct 2, 2024

x0.cockpit.lan is the name of the local machine, but specifying it with login_and_go will navigate to it as a remote host, using a URL like

https://127.0.0.2:9091/@x0.cockpit.lan

This is not needed since the test only wants the session on localhost, and doesn't care what happens inside the session. It would be harmless except that Cockpit might spontaneously open the "Add host" dialog for "x0.cockpit.lan", which will make the session menu inaccessible for the subsequent logout. (Only on Firefox, Chrome manages to open it anyway.)

x0.cockpit.lan is the name of the local machine, but specifying it
with login_and_go will navigate to it as a remote host, using a URL
like

    https://127.0.0.2:9091/@x0.cockpit.lan

This is not needed since the test only wants the session on localhost,
and doesn't care what happens inside the session. It would be harmless
except that Cockpit might spontaneously open the "Add host" dialog for
"x0.cockpit.lan", which will make the session menu inaccessible for
the subsequent logout. (Only on Firefox, Chrome manages to open it
anyway.)
Copy link
Member

@martinpitt martinpitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup!

@mvollmer
Copy link
Member Author

mvollmer commented Oct 2, 2024

Cockpit might spontaneously open the "Add host" dialog for "x0.cockpit.lan",

This only starts happening with #20826 or #21012, I think...

@mvollmer
Copy link
Member Author

mvollmer commented Oct 2, 2024

Cockpit might spontaneously open the "Add host" dialog for "x0.cockpit.lan",

This only starts happening with #20826 or #21012, I think...

No, seems to be new with #21012, which probably has changed the timing... anyway, it's a good fix. :-)

@mvollmer mvollmer merged commit 7afb886 into cockpit-project:main Oct 2, 2024
86 checks passed
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.

2 participants