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.

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

OAUTH2-210 Using wrong clientId with Authorization Code Flow Grant #61517

Closed
wants to merge 6 commits into from

Conversation

@topolik
Copy link

topolik commented Jul 20, 2018

martamedio and others added 6 commits Jul 17, 2018
…nt ID
…t to avoid LiferayOAuthDataProvider WARN
@liferay-continuous-integration

This comment has been minimized.

Copy link
Collaborator

liferay-continuous-integration commented Jul 20, 2018

CI is automatically triggering "ci:test:sf" and "ci:test:relevant" for this pull to run Source Formatter and relevant tests.

Comment "ci:test" to run the full PR Tester for this pull.

@topolik topolik changed the title pull-request-571 OAUTH2-210 Using wrong clientId with Authorization Code Flow Grant Jul 20, 2018
@liferay-continuous-integration

This comment has been minimized.

Copy link
Collaborator

liferay-continuous-integration commented Jul 20, 2018

✔️ ci:test:sf - 1 out of 1 jobs passed in 2 minutes 2 seconds 187 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 761fa543339c25a7c170e87945c8a5e565ddfc87

Sender Branch:

Branch Name: pull-request-571
Branch GIT ID: f0d92c16b53fcc7cb5a6ed9c00a5c9f16020e521

1 out of 1jobs PASSED
1 Successful Jobs:
For more details click here.
@liferay-continuous-integration

This comment has been minimized.

Copy link
Collaborator

liferay-continuous-integration commented Jul 20, 2018

✔️ ci:test:relevant - 23 out of 23 jobs passed in 1 hour 2 minutes 36 seconds 691 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 761fa543339c25a7c170e87945c8a5e565ddfc87

Copied in Private Modules Branch:

Branch Name: master-private
Branch GIT ID: 789a1ebf5f39d40a50ff47c11ad2887eda457f75

23 out of 23 jobs PASSED
23 Successful Jobs:
For more details click here.
@brianchandotcom

This comment has been minimized.

Copy link
Owner

brianchandotcom commented Jul 20, 2018

Merged. Thank you.
View total diff: 4b78477...9699c3d

@brianchandotcom

This comment has been minimized.

Copy link
Owner

brianchandotcom commented Jul 20, 2018

@topolik please see my changes in upstream. Thx.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.