Skip to content

remove protocol-relative / implicit#421

Merged
Oumph merged 1 commit intomainfrom
no_relative
Apr 12, 2026
Merged

remove protocol-relative / implicit#421
Oumph merged 1 commit intomainfrom
no_relative

Conversation

@Oumph
Copy link
Copy Markdown
Member

@Oumph Oumph commented Apr 12, 2026

@Oumph Oumph self-assigned this Apr 12, 2026
@Oumph Oumph requested a review from Trim April 12, 2026 12:18
@Trim
Copy link
Copy Markdown
Member

Trim commented Apr 12, 2026

Edit 1: Oups, j'ai mal utilisé les lignes de commande, ce commentaire est faux... désolé

Merci, si jamais, ça va entrer en conflit avec #399 , mais ça sera facile à corriger (il faudra garder la version présente dans la branche de #399)

Je vois qu'il reste des références en dur à http:// (schéma non-TLS) également dans:

  • app/controllers/admin/stylesheets_controller

Il reste également des liens sans schémas dans:

  • app/helpers/atom_helper.rb
  • app/helpers/avatar_uploader.rb
  • app/views/stylesheets/edit.html.haml

Edit2: bon, j'ai repris correctement les modifications et je vois qu'il reste uniquement app/controllers/admin/stylesheets_controller qui utilise le schéma http://. J'y mettrai bien le schéma https:// pour sécuriser la connexion de phantomjs parce qu'il semble transmettre le cookie de session.

@Oumph Oumph merged commit d1788ac into main Apr 12, 2026
@Oumph Oumph deleted the no_relative branch April 12, 2026 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants