Skip to content

Solves the problem of Flask-User missing actual runtime information#64

Open
jcborras wants to merge 2 commits intolingthio:masterfrom
jcborras:master
Open

Solves the problem of Flask-User missing actual runtime information#64
jcborras wants to merge 2 commits intolingthio:masterfrom
jcborras:master

Conversation

@jcborras
Copy link
Copy Markdown

@jcborras jcborras commented Feb 3, 2015

when working behind a reverse proxy (i.e. nginx serving static content
or handling HTTPS traffic).

  • by adding a USER_APP_ROOT_URL customization value
  • by generating USER_APP_ROOT_URL + url_for(, _external=True) in
    links when the customization value is available
  • by adding USER_APP_ROOT_URL + url_for(, _external=True) in
    emails when the customization value is available

when working behind a reverse proxy (i.e. nginx serving static content
or handling HTTPS traffic).

  - by adding a USER_APP_ROOT_URL customization value
  - by generating USER_APP_ROOT_URL + url_for(, _external=True) in
    links when the customization value is available
  - by adding USER_APP_ROOT_URL  + url_for(, _external=True) in
    emails when the customization value is available
@and-semakin
Copy link
Copy Markdown
Collaborator

Hi @jcborras! Thank you for your contribution! I'm sorry you had to wait for an answer for so long. Could you please rebase your branch over actual master?

@and-semakin and-semakin added the Rebase Requested Maintainer has requested a contributor to update/rebase PR branch over actual master branch label Nov 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Rebase Requested Maintainer has requested a contributor to update/rebase PR branch over actual master branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants