authentication-flows icon indicating copy to clipboard operation
authentication-flows copied to clipboard

better url route handling

Open ggmartins opened this issue 9 years ago • 1 comments

The modifications here were required to make auth-flows work with tomcat+apache ProxyPass.

ggmartins avatar Jul 23 '15 20:07 ggmartins

Hi

Thanks for your contribution, much appreciated! A while back, I've sent you a request to change several things in your Pull Request, so I can accept it. Do you plan to do these changes? Do you use the auth-flows in production? Please share a link so I can see it working :-)

thanks Ohad


Ohad Redlich [image: linked-in with me] http://il.linkedin.com/in/ohadredlich/[image: Join my Facebook Page] https://www.facebook.com/ohadr.photography https://github.com/OhadR http://stackoverflow.com/users/421642/ohadr

On Thu, Jul 23, 2015 at 11:37 PM, ggmartins [email protected] wrote:

The modifications here were required to make auth-flows work with

tomcat+apache ProxyPass.

You can view, comment on, or merge this pull request online at:

https://github.com/OhadR/oAuth2-sample/pull/67 Commit Summary

  • added loginSuccessEndpointUrl
  • changePassword fix

File Changes

  • M authentication-flows/src/main/java/com/ohadr/auth_flows/config/AuthFlowsProperties.java https://github.com/OhadR/oAuth2-sample/pull/67/files#diff-0 (16)
  • M authentication-flows/src/main/java/com/ohadr/auth_flows/core/AuthenticationFlowsProcessorImpl.java https://github.com/OhadR/oAuth2-sample/pull/67/files#diff-1 (40)
  • M authentication-flows/src/main/java/com/ohadr/auth_flows/core/AuthenticationSuccessHandler.java https://github.com/OhadR/oAuth2-sample/pull/67/files#diff-2 (9)
  • M authentication-flows/src/main/java/com/ohadr/auth_flows/interfaces/AuthenticationFlowsProcessor.java https://github.com/OhadR/oAuth2-sample/pull/67/files#diff-3 (4)
  • M authentication-flows/src/main/java/com/ohadr/auth_flows/types/FlowsConstatns.java https://github.com/OhadR/oAuth2-sample/pull/67/files#diff-4 (4)
  • M authentication-flows/src/main/java/com/ohadr/auth_flows/web/UserActionController.java https://github.com/OhadR/oAuth2-sample/pull/67/files#diff-5 (36)
  • M authentication-flows/src/test/resources/client.properties https://github.com/OhadR/oAuth2-sample/pull/67/files#diff-6 (2)

Patch Links:

  • https://github.com/OhadR/oAuth2-sample/pull/67.patch
  • https://github.com/OhadR/oAuth2-sample/pull/67.diff

— Reply to this email directly or view it on GitHub https://github.com/OhadR/oAuth2-sample/pull/67.

OhadR avatar Oct 07 '15 14:10 OhadR