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

"After changing password, can log in with new password" tries to login with an access_token #954

Closed
Half-Shot opened this issue Sep 16, 2020 · 1 comment · Fixed by #955
Labels
T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.

Comments

@Half-Shot
Copy link
Contributor

Half-Shot commented Sep 16, 2020

test "After changing password, can log in with new password",
requires => [ local_user_fixture( password => $password ) ],
check => sub {
my ( $user ) = @_;
matrix_set_password( $user, $password, "my new password" )->then( sub {
do_request_json_for( $user,
method => "POST",
uri => "/r0/login",
content => {
type => "m.login.password",
user => $user->user_id,
password => "my new password",
}
);
})->then_done(1);
};

I don't think it should be trying to do that :(

Logs can be found on https://buildkite.com/matrix-dot-org/synapse/builds/11896#

@Half-Shot Half-Shot added the T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues. label Sep 16, 2020
@erikjohnston
Copy link
Member

The access token will get auto added by do_request_json_for. You should be able to change it to $user->http->do_request_json( ... )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants