No need to manually force port 80 when X-Forwarded-Port is not specif…

spring-attic / spring-social

Allows you to connect your applications with SaaS providers such as Facebook and Twitter.

http://projects.spring.io/spring-social

Apache License 2.0

619 stars 351 forks source link

No need to manually force port 80 when X-Forwarded-Port is not specif… #259

Open codeconsole opened 6 years ago

codeconsole commented 6 years ago

…ied. This results in a bug when having X-Forwarded-Proto as https and X-Forwarded-Port not specified. It will produce a URL with https and port 80 appended at the end. This bug is prevelant on the most current App Engine Java Flexible environment. It specified the scheme, but no port which causes the code to fail and redirect to a non-existant url.

codeconsole commented 5 years ago

@habuma please merge

codeconsole commented 5 years ago

@habuma