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.

Sign up for GitHub

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

Jump to bottom

Add authn_method to Consumer.complete() #852

Merged
merged 1 commit into from
Mar 27, 2023
Merged

Add authn_method to Consumer.complete() #852

merged 1 commit into from
Mar 27, 2023

Conversation

stinovlas
Copy link
Member

@stinovlas stinovlas commented Mar 24, 2023 •
edited
Loading

  • Any changes relevant to users are recorded in the CHANGELOG.md.
  • The documentation has been updated, if necessary.
    • Documentation wasn't updated, because there is no relevant documentation about Consumer.complete at the moment.
  • New code is annotated.
  • Changes are covered by tests.
    • I didn't add any test because it's just an argument that's passed to the underlying function and it should already be tested there.

Closes #851

@stinovlas stinovlas force-pushed the 851-add-consumer-complete-authn-method branch from d01d018 to 1d46f09 Compare March 24, 2023 07:49
@codecov-commenter
Copy link

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (a8454e3) 63.82% compared to head (1d46f09) 63.82%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files 
@@           Coverage Diff           @@
##           master     #852   +/-   ##
=======================================
  Coverage   63.82%   63.82%           
=======================================
  Files          64       64           
  Lines       11877    11877           
  Branches     2154     2154           
=======================================
  Hits         7581     7581           
  Misses       3693     3693           
  Partials      603      603
Impacted Files Coverage Δ
src/oic/oic/consumer.py 79.92% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

schlenk
schlenk approved these changes Mar 24, 2023
View reviewed changes
Copy link
Collaborator

@schlenk schlenk left a comment

Choose a reason for hiding this comment

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

LGTM

@tpazderka tpazderka merged commit 3b1a9c3 into master Mar 27, 2023
@tpazderka tpazderka deleted the 851-add-consumer-complete-authn-method branch March 27, 2023 07:24
infohash added a commit to infohash/pyoidc that referenced this pull request Jul 2, 2023
@infohash
Squashed commit of the following:
a1829af 
commit 5245dca
Author: tpazderka <tomas.pazderka@nic.cz>
Date:   Tue May 16 19:48:56 2023 +0200

    Fix building of the documentation (CZ-NIC#860)

commit ba65e00
Author: Tomáš Pazderka <tomas.pazderka@nic.cz>
Date:   Mon May 15 09:01:02 2023 +0200

    Prepare 1.6.0 release

commit cfd0b6e
Author: Michael Schlenker <MichaelSchlenker@gmx.net>
Date:   Tue May 2 14:02:37 2023 +0200

    Repair some oauth_examples and get rid of the outdated jquery file (CZ-NIC#857)

    * More cleanup

    * Add oauth_example to quality checks

    Signed-off-by: Michael Schlenker <michael.schlenker@contact-software.com>

    * Update CHANGELOG.md

    ---------

    Signed-off-by: Michael Schlenker <michael.schlenker@contact-software.com>
    Co-authored-by: Michael Schlenker <michael.schlenker@contact-software.com>

commit 444bd68
Merge: 3b1a9c3 433017c
Author: tpazderka <tomas.pazderka@nic.cz>
Date:   Wed Apr 5 13:59:40 2023 +0200

    Merge pull request CZ-NIC#854 from gbip/853_session_state

    Use session_state to backup consumer state if available

commit 433017c
Author: Paul Florence <paul.florence@makina-corpus.com>
Date:   Mon Apr 3 17:37:30 2023 +0200

    Use 'session_state' from an Authentication Response to backup consumer state (if available)

commit 3b1a9c3
Merge: a8454e3 1d46f09
Author: tpazderka <tomas.pazderka@nic.cz>
Date:   Mon Mar 27 09:24:34 2023 +0200

    Merge pull request CZ-NIC#852 from CZ-NIC/851-add-consumer-complete-authn-method

    Add authn_method to Consumer.complete()

commit 1d46f09
Author: Jan Musílek <jan.musilek@nic.cz>
Date:   Thu Mar 23 16:25:48 2023 +0100

    Add authn_method to Consumer.complete()

commit a8454e3
Merge: 559c9c8 a855642
Author: tpazderka <tomas.pazderka@nic.cz>
Date:   Fri Mar 17 13:16:25 2023 +0100

    Merge pull request CZ-NIC#850 from hlin/master

    Correct OpenID Connect Core specification URL in doc

commit a855642
Author: Hypo Lin <hlin@hypo.name>
Date:   Fri Mar 17 18:56:19 2023 +0800

    Correct OpenID Connect Core specification URL in doc

commit 559c9c8
Merge: 76debd6 9670f83
Author: tpazderka <tomas.pazderka@nic.cz>
Date:   Mon Mar 13 16:06:25 2023 +0100

    Merge pull request CZ-NIC#847 from CZ-NIC/use-pydantic

    Improve settings by using pydantic

commit 9670f83
Author: Tomáš Pazderka <tomas.pazderka@nic.cz>
Date:   Mon Feb 20 12:42:46 2023 +0100

    Improve settings by using pydantic

commit 76debd6
Merge: 6f7f893 94d1f99
Author: tpazderka <tomas.pazderka@nic.cz>
Date:   Mon Mar 13 14:54:53 2023 +0100

    Merge pull request CZ-NIC#848 from CZ-NIC/bandit-timeouts

    Pass timeout to all requests

commit 94d1f99
Author: Tomáš Pazderka <tomas.pazderka@nic.cz>
Date:   Fri Mar 10 10:19:00 2023 +0100

    Pass timeout to all requests

commit 6f7f893
Merge: f6c590c 12247bc
Author: tpazderka <tomas.pazderka@nic.cz>
Date:   Mon Feb 20 09:38:17 2023 +0100

    Merge pull request CZ-NIC#846 from CZ-NIC/repo-move

    Fix links after repo move

commit 12247bc
Author: Tomáš Pazderka <tomas.pazderka@nic.cz>
Date:   Fri Feb 17 11:50:41 2023 +0100

    Fix links after repo move

    Close CZ-NIC#845
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@schlenk schlenk schlenk approved these changes

@tpazderka tpazderka tpazderka approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Allow using different authn_method in Consumer.complete()
4 participants
@stinovlas @codecov-commenter @schlenk @tpazderka