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

Inform users that Force HTTPS config option will also force the usage of secure cookies #16511

Merged
merged 1 commit into from
Jun 6, 2017

Conversation

andrepereiradasilva
Copy link
Contributor

@andrepereiradasilva andrepereiradasilva commented Jun 4, 2017

Pull Request for New Issue.

Summary of Changes

When we enabled "Force HTTPS" Joomla will also force the use of secure cookies.
The user currently is not informed of this.

Why the proposed change:
We can have several ways of forcing HTTPS in a website, for instance, at web server level, ie, without even using Joomla "Force HTTPS" option (leaving it to "None").

But forcing http to https redirect at web server level will not force the cookies to be secure, Joomla "Force HTTPS" config option force the use of https AND forces the use of secure session cookies, so IMHO is relevant for server admin to know that.

Testing Instructions

Language review.

Expected result

User informed that site will also use secure cookies.

Actual result

Not informed.

Documentation Changes Required

None.

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-staging labels Jun 4, 2017
@richard67
Copy link
Member

I have tested this item ✅ successfully on f1350f3

The changed text is shown in the tool tip for the "Force HTTPS" value when this PR is applied.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16511.

@Quy
Copy link
Contributor

Quy commented Jun 5, 2017

I have tested this item ✅ successfully on f1350f3


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16511.

@joomla-cms-bot joomla-cms-bot removed Language Change This is for Translators PR-staging labels Jun 5, 2017
@ghost
Copy link

ghost commented Jun 5, 2017

RTC after two successful tests.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jun 5, 2017
@brianteeman brianteeman added the Language Change This is for Translators label Jun 6, 2017
@wilsonge wilsonge merged commit 6e58fb8 into joomla:staging Jun 6, 2017
@joomla-cms-bot joomla-cms-bot added PR-staging and removed RTC This Pull Request is Ready To Commit labels Jun 6, 2017
@wilsonge wilsonge added this to the Joomla 3.7.3 milestone Jun 6, 2017
@andrepereiradasilva andrepereiradasilva deleted the patch-27 branch June 6, 2017 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language Change This is for Translators
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants