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

HTTPS: mixed content en production #14

Closed
JoeCrescoll opened this issue Oct 14, 2018 · 7 comments
Closed

HTTPS: mixed content en production #14

JoeCrescoll opened this issue Oct 14, 2018 · 7 comments
Projects

Comments

@JoeCrescoll
Copy link
Collaborator

Description

Malgré l'utilisation du HTTPS dans la variable d'environnent APP_URLen production, la fonction asset() de Laravel semble retourner des chemins avec du HTTP, le navigateur bloque donc certains assets.

Exemple avec mixed content bloqué

Une bonne solution semble être expliquée ici.

@greut
Copy link
Member

greut commented Oct 15, 2018

asset(..., true); une solution du pauvre[1]. Sinon, il faut regarder pour les entêtes HTTP, tel que X-Forwarded-Proto car il est possible qu'ils n'arrivent pas jusqu'à l'application PHP. (problème avancé)

  1. https://github.com/illuminate/routing/blob/master/UrlGenerator.php#L223

@JoeCrescoll
Copy link
Collaborator Author

JoeCrescoll commented Oct 15, 2018

Hello @greut,

Merci pour ces infos !
Je ne m'étais pas rendu compte que le container tourne derrière un proxy.

En faisant quelques recherches, je suis rapidement tombé sur ce lien et celui-là. Je vais tester ça un de ces quatre.
L'adresse du proxy peut s'obtenir en regardant les logs d'accès du serveur Nginx j'imagine ?

@greut
Copy link
Member

greut commented Oct 16, 2018

Il y a en réalité deux proxies, traefik qui fournit le certificat TLS et l'adresse publique ainsi qu'un nginx local qui sert les fichiers statiques et parle avec php-fpm. C'est un peu tordu mais c'était pour vous permettre de jouer avec nginx.

@JoeCrescoll
Copy link
Collaborator Author

Merci pour ces explications et pour la piste sur les entêtes X-Forwarded-* 👍

@JoeCrescoll JoeCrescoll added this to In progress in MenuFinder Oct 16, 2018
@greut
Copy link
Member

greut commented Oct 17, 2018

@JoeCrescoll c'est un problème que vous héritez de l'architecture en place. Ca pourrait être décrit dans le README de la partie laravel d'ailleurs.

https://github.com/HE-Arc/webapp-server/tree/master/files/laravel

vos camarades et/ou @grunenwald vous seront certainement très reconnaissants d'y contribuer une solution simple.

@grunenwald
Copy link
Member

Oui, je confirme : une reconnaissance infinie ! ;)

@JoeCrescoll
Copy link
Collaborator Author

Aucun problème, je ferai une pull request sur HE-Arc/webapp-server d'ici ce soir.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
MenuFinder
  
Done
Development

No branches or pull requests

3 participants