Problem/Motivation

At #3103653: Inject the current request in the OAuth2Manager, we have already injected the request object, so any implementer can accessed the request information.

Proposed resolution

Use the parent's request object and remove the local request object from the class

CommentFileSizeAuthor
#2 3119944-2.patch1.85 KBgvso

Comments

gvso created an issue. See original summary.

gvso’s picture

gvso
Primary language Spanish
commented
Status: Active » Needs review
StatusFileSize
new 3119944-2.patch1.85 KB

Status: Needs review » Needs work

The last submitted patch, 2: 3119944-2.patch, failed testing. View results

gvso’s picture

gvso
Primary language Spanish
commented
Status: Needs work » Needs review
davps’s picture

davps commented
Assigned: Unassigned » davps

  • davps committed 5ff4d358 on 8.x-2.x 
    Issue #3119944 by gvso, davps: Use parent's request object in...
davps’s picture

davps commented
Assigned: davps » Unassigned
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.