Default Project CR-7612

OAuth2 resource owner authentication code does not pass headers to auth module

Closed on 02 Sep 15

  •  
  •  
  •  
  •  
  • Author & Moderator
  • Reviewers
    • Reviewer completed
    • Reviewer completed

CR-7612 2

Keyboard shortcuts  
Summarize the review outcomes (optional)
 
#permalink

Details

Warning: no files are visible, they have all been filtered.
Participant Role Time Spent Comments Latest Comment
Author & Moderator 17m 1 Sorry. I forgot to pull the commented original line out. ...
Reviewer - Complete 2m 1 Can you also add someone from support to review this as w...
Reviewer - Complete 19m    
Total   38m 2  
#permalink

Objectives

The OAuth2 resource owner auth code does not pass Request or Response to AuthContext login method.

#permalink

Issues Raised From Comments

Key Summary State Assignee
#permalink

General Comments

21 Aug 15

phillcunnington says:

Can you also add someone from support to review this as well if its going to be committed to 12.0.0.
Also if we are going to switch the login() method then we just remove the old one not just comment it out.

27 Aug 15

sfraser says:

Sorry. I forgot to pull the commented original line out. It will not be included in 12.0.x or trunk

/Users/.../oauth2/OpenAMResourceOwnerAuthenticator.java Changed  
Open in IDE #permalink
/OPENAM-7395.diff Added
Open in IDE #permalink

Review updated: Reload | Ignore | Collapse

You cannot reload the review while writing a comment.

Create Issue

X
Assign To Me

Log time against