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

Update README.md #48

Merged
merged 1 commit into from
Feb 25, 2015
Merged

Update README.md #48

merged 1 commit into from
Feb 25, 2015

Conversation

antonmedv
Copy link
Contributor

Something like this.

Something like this.
@phanan
Copy link
Owner

phanan commented Feb 25, 2015

Shouldn't these 3 lines

RewriteEngine on
RewriteCond %{HTTPS} !on
RewriteRule (.*) https://%{HTTP_HOST}%{REQUEST_URI}

be kept?

@antonmedv
Copy link
Contributor Author

Not sure, but think Strict-Transport-Security must be on every HTTP request. It's better to ask some one else on StackExchange. 😄

@phanan
Copy link
Owner

phanan commented Feb 25, 2015

Apparently this can and should be set in the <VirtualHost> directive instead, no?

On Wed, Feb 25, 2015 at 10:01 PM, Anton Medvedev [email protected]
wrote:

Not sure, but think Strict-Transport-Security must be on every HTTP
request. It's better to ask some one else on StackExchange. [image:
😄]


Reply to this email directly or view it on GitHub
#48 (comment).

@phanan phanan merged commit d34ec0c into phanan:master Feb 25, 2015
phanan pushed a commit that referenced this pull request Feb 25, 2015
@antonmedv antonmedv deleted the patch-1 branch February 26, 2015 07:16
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