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

Upload & install plugin ZIP fails with "Wrong username and/or password." #21274

Closed
mattab opened this issue Sep 18, 2023 · 3 comments · Fixed by #21278
Closed

Upload & install plugin ZIP fails with "Wrong username and/or password." #21274

mattab opened this issue Sep 18, 2023 · 3 comments · Fixed by #21278
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@mattab
Copy link
Member

mattab commented Sep 18, 2023

In Matomo 5 RC, trying to upload a plugin ZIP of a plugin:
Screenshot from 2023-09-19 06-44-55

And entering the correct password,

then it fails with

An error occurred
Wrong username and/or password.

image

@mattab mattab added Regression Indicates a feature used to work in a certain way but it no longer does even though it should. Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. To Triage An issue awaiting triage by a Matomo core team member labels Sep 18, 2023
@mattab mattab added this to the 5.0.0 milestone Sep 18, 2023
@sgiehl
Copy link
Member

sgiehl commented Sep 18, 2023

@mattab did you test that on Matomo 4 and does your password contain a char like & ?

@sgiehl sgiehl added Waiting for user feedback Indicates the Matomo team is waiting for feedback from the author or other users. and removed To Triage An issue awaiting triage by a Matomo core team member labels Sep 18, 2023
@mattab mattab changed the title Upload & install plugin ZIP fails with Upload & install plugin ZIP fails with " Wrong username and/or password." Sep 19, 2023
@mattab mattab changed the title Upload & install plugin ZIP fails with " Wrong username and/or password." Upload & install plugin ZIP fails with "Wrong username and/or password." Sep 19, 2023
@mattab
Copy link
Member Author

mattab commented Sep 19, 2023

@sgiehl reproduced on 5.0.0-rc2 so it's a regression I think?

@sgiehl
Copy link
Member

sgiehl commented Sep 19, 2023

Technically not a regression as the problem existed there before. The upload mechanism has it's own password check, which still uses Common::getRequestVar to receive the value. As it doesn't unsanitize the value, every char that might get escaped causes problems there. I'll prepare a PR to fix that, as it's easy to do.

@sgiehl sgiehl added Bug For errors / faults / flaws / inconsistencies etc. and removed Regression Indicates a feature used to work in a certain way but it no longer does even though it should. Waiting for user feedback Indicates the Matomo team is waiting for feedback from the author or other users. Potential Bug Something that might be a bug, but needs validation and confirmation it can be reproduced. labels Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants