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

ROLE_CIDE_#inpector - Password change #64

Open
MartinTuchyna opened this issue Dec 29, 2019 · 2 comments
Open

ROLE_CIDE_#inpector - Password change #64

MartinTuchyna opened this issue Dec 29, 2019 · 2 comments
Labels
out_of_scope Requirements that are not within the CIDE APP specification! testing

Comments

@MartinTuchyna
Copy link
Member

If the user changes the password it is possible to login with the old password. The new password also works but there is no point in leaving the possibility of using the old password. We need to change the password two times in a row with the same new password to disable the old one.

@klimeto klimeto added the bug Something isn't working label Dec 29, 2019
@klimeto
Copy link
Collaborator

klimeto commented Dec 30, 2019

After some hours of testing the CIDE APP code, it seems the problem is not on the CIDE APP, but on the authentication service provided by Geoserver.

It seems Geoserver is caching the old password for some time after its change, and so for this period username can login with both passwords, the old and the new one.

After changing login with both worked as follows:

NEW PASSWORD:

Request URL: https://pproo.azo.hr/bifisic/services/httpbasicauth/auth?user=klimetom_admin&password=kkkkkkkk
Request Method: GET
Status Code: 200 OK
Remote Address: 213.202.126.157:443
Referrer Policy: no-referrer-when-downgrade
Connection: Keep-Alive
Content-Type: application/json;charset=UTF-8
Date: Mon, 30 Dec 2019 21:42:28 GMT
Keep-Alive: timeout=5, max=100
Server: Apache/2.4.6 (CentOS) OpenSSL/1.0.2k-fips PHP/5.4.16 mod_wsgi/3.4 Python/2.7.5
Transfer-Encoding: chunked

OLD PASSWORD:

Request URL: https://pproo.azo.hr/bifisic/services/httpbasicauth/auth?user=klimetom_admin&password=aaaaaaaa
Request Method: GET
Status Code: 200 OK
Remote Address: 213.202.126.157:443
Referrer Policy: no-referrer-when-downgrade
Connection: Keep-Alive
Content-Type: application/json;charset=UTF-8
Date: Mon, 30 Dec 2019 21:43:41 GMT
Keep-Alive: timeout=5, max=100
Server: Apache/2.4.6 (CentOS) OpenSSL/1.0.2k-fips PHP/5.4.16 mod_wsgi/3.4 Python/2.7.5
Transfer-Encoding: chunked

OLD PASSWORD (after cca 5 mins)

Request URL: https://pproo.azo.hr/bifisic/services/httpbasicauth/auth?user=klimetom_admin&password=aaaaaaaa
Request Method: GET
Status Code: 401 Unauthorized
Remote Address: 213.202.126.157:443
Referrer Policy: no-referrer-when-downgrade
Connection: Keep-Alive
Content-Type: application/json;charset=UTF-8
Date: Mon, 30 Dec 2019 21:45:22 GMT
Keep-Alive: timeout=5, max=100
Server: Apache/2.4.6 (CentOS) OpenSSL/1.0.2k-fips PHP/5.4.16 mod_wsgi/3.4 Python/2.7.5
Transfer-Encoding: chunked

And now also CIDE authenticates the user onlyh with new password:

image

This issue is linked to the bifisic authorization service provided by Geoserver and needs to be tackled by Fabrizio his team.

Otherwise the user after the password change can still work in CIDE and after a short period of time, the next time he will be denied with the new password.

By my tests it the old password is invalid after about 10 minutes.

@klimeto klimeto added out_of_scope Requirements that are not within the CIDE APP specification! and removed bug Something isn't working labels Dec 30, 2019
@klimeto
Copy link
Collaborator

klimeto commented Dec 30, 2019

I made another test where I tried to login directly to Geoserver Web Admin console and there it DID NOT work as described above.

It seems the problem resides on the BIFISIC AUTH service deployed by Fabrizio available from this URL https://pproo.azo.hr/bifisic/services/httpbasicauth/auth

After 10 minutes the old password returns 401 Unauthorised, but before user can login with both old and new password!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
out_of_scope Requirements that are not within the CIDE APP specification! testing
Projects
None yet
Development

No branches or pull requests

2 participants