Projet

Général

Profil

0001-misc-remove-you-have-been-logged-out-message-33703.patch

Frédéric Péters, 05 juin 2019 18:35

Télécharger (2,67 ko)

Voir les différences:

Subject: [PATCH] misc: remove "you have been logged out" message (#33703)

 src/authentic2/views.py | 1 -
 tests/test_auth_oidc.py | 4 ++--
 tests/test_idp_oidc.py  | 4 ++--
 3 files changed, 4 insertions(+), 5 deletions(-)
src/authentic2/views.py
586 586
    response = utils.redirect(request, next_url)
587 587
    if local_logout_done:
588 588
        response.set_cookie('a2_just_logged_out', 1, max_age=60)
589
        messages.info(request, _('You have been logged out'))
590 589
    return response
591 590

  
592 591

  
tests/test_auth_oidc.py
395 395
    with utils.check_log(caplog, 'revoked token from OIDC'):
396 396
        with oidc_provider_mock(oidc_provider, oidc_provider_jwkset, code):
397 397
            response = response.click(href='logout')
398
    assert 'https://idp.example.com/logout' in response.text
398
    assert response.location.startswith('https://idp.example.com/logout?')
399 399

  
400 400

  
401 401
def test_show_on_login_page(app, oidc_provider):
......
461 461
    with utils.check_log(caplog, 'revoked token from OIDC'):
462 462
        with oidc_provider_mock(oidc_provider, oidc_provider_jwkset, code, nonce=nonce):
463 463
            response = response.click(href='logout')
464
    assert 'https://idp.example.com/logout' in response.text
464
    assert response.location.startswith('https://idp.example.com/logout?')
465 465

  
466 466

  
467 467
def test_register_issuer(db, app, caplog, oidc_provider_jwkset):
tests/test_idp_oidc.py
328 328
        }
329 329
        logout_url = make_url('oidc-logout', params=params)
330 330
        response = app.get(logout_url)
331
        assert 'You have been logged out' in response.text
332
        assert 'https://example.com/?state=xyz' in response.text
331
        assert response.status_code == 302
332
        assert response.location == 'https://example.com/?state=xyz'
333 333
        assert '_auth_user_id' not in app.session
334 334
    else:
335 335
        response = app.get(make_url('account_management'))
336
-