GluuFederation / gluu-passport

Gluu interface to Passport.js to support social login and inbound identity.
Apache License 2.0
6 stars 16 forks source link

fix(openid-client): pass provided scope in auth request #539

Closed kdhttps closed 1 year ago

kdhttps commented 1 year ago

PR Contains: openid-client lib not really taking its client object scope so need to pass it from config

closes #538

codecov[bot] commented 1 year ago

Codecov Report

Merging #539 (7c5e1a8) into master (9c9e274) will increase coverage by 0.08%. The diff coverage is 100.00%.

:exclamation: Current head 7c5e1a8 differs from pull request most recent head a65c310. Consider uploading reports for the commit a65c310 to get more accurate results

@@            Coverage Diff             @@
##           master     #539      +/-   ##
==========================================
+ Coverage   78.79%   78.87%   +0.08%     
==========================================
  Files          35       35              
  Lines         778      781       +3     
==========================================
+ Hits          613      616       +3     
  Misses        165      165              
Impacted Files Coverage Δ
config/test.js 100.00% <ø> (ø)
server/routes.js 64.28% <100.00%> (+1.32%) :arrow_up:
sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication