Skip to content

Use ConnectionBuilder to call userinfo endpoint - #468

Merged
agologan merged 1 commit into
openid:masterfrom
madurangasiriwardena:user-info
Jun 8, 2021
Merged

Use ConnectionBuilder to call userinfo endpoint#468
agologan merged 1 commit into
openid:masterfrom
madurangasiriwardena:user-info

Conversation

@madurangasiriwardena

@madurangasiriwardena madurangasiriwardena commented May 20, 2019

Copy link
Copy Markdown
Contributor

Checklist

  • I read the Contribution Guidelines
  • I signed the CLA and WG Agreements
  • I ran, updated and added unit tests as necessary.
  • I verified the contribution matches existing coding style.
  • I updated the documentation if necessary.

Motivation and Context

Userinfo endpoint call does not honor https_required configuration, which is done for token endpoint call.

Resolves #467

@codecov-io

codecov-io commented May 20, 2019

Copy link
Copy Markdown

Codecov Report

Merging #468 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #468   +/-   ##
=========================================
  Coverage     83.71%   83.71%           
  Complexity      479      479           
=========================================
  Files            42       42           
  Lines          2345     2345           
  Branches        233      233           
=========================================
  Hits           1963     1963           
  Misses          299      299           
  Partials         83       83

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f125925...cc5a7b6. Read the comment docs.

@madurangasiriwardena

Copy link
Copy Markdown
Contributor Author

Any update on this?

@agologan

agologan commented Jun 8, 2021

Copy link
Copy Markdown
Collaborator

LGTM

@agologan
agologan merged commit c9f76f1 into openid:master Jun 8, 2021
@cumcum8197

Copy link
Copy Markdown

c9f76f1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Userinfo endpoint call does not honor https_required configuration

4 participants