DiscourseConnect genera redirect HTTP anche con force_https attivo

[supermathie (Michael Brown)]

@matiasgarciaisaia

dici al mio Traefik di aggiungere un header X-Forwarded-Proto: https

Questo è come lo facciamo sulla nostra piattaforma di hosting; abbiamo uno strato di bilanciamento del carico che imposta X-Forwarded-Proto per il downstream nginx+Discourse da consumare.

Non abbiamo bisogno di trucchi aggiuntivi per farlo funzionare: non sono sicuro di cosa stia andando storto per te qui.

@matiasgarciaisaia

Ancora una volta, penso che se c’è un’impostazione force_https, Discourse-the-rails-app dovrebbe rispettarla, indipendentemente da ciò che il proxy inverso o altre parti gestiscono o meno.

Questo è infatti ciò che accade:

  def self.generate_sso(return_path = "/", secure_session:)
    sso = new(secure_session: secure_session)
    sso.nonce = SecureRandom.hex
    sso.register_nonce(return_path)
    sso.return_sso_url = Discourse.base_url + "/session/sso_login"
    sso
  end

e base_url proviene da:

  def self.base_protocol
    SiteSetting.force_https? ? "https" : "http"
  end

  def self.base_url_no_prefix
    "#{base_protocol}://#{current_hostname_with_port}"
  end

  def self.base_url
    base_url_no_prefix + base_path
  end