Skip to content

Commit

Permalink
Remove HTTPS header as it cannot be validated
Browse files Browse the repository at this point in the history
  • Loading branch information
maximejobin authored Sep 5, 2016
1 parent b41e115 commit a6c3bfa
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions rocket-nginx.conf
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,6 @@ location ~ /wp-content/cache/wp-rocket/.*html$ {
expires 30d;

#!# HEADER_HTTP #!#
#!# HEADER_HTTPS #!#
#!# HEADER_NON_GZIP #!#
}

Expand All @@ -156,7 +155,6 @@ location ~ /wp-content/cache/wp-rocket/.*_gzip$ {
expires 30d;

#!# HEADER_HTTP #!#
#!# HEADER_HTTPS #!#
#!# HEADER_GZIP #!#
}

Expand Down

4 comments on commit a6c3bfa

@CreaoticX
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you explain this? I'm going to setup my site as https only, but this is the first time I've done an nginx server without apache. Not sure what removing that does. Thanks.

@CreaoticX
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK I found this. #38 Got it. I'll work on the issue.

@maximejobin
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm almost done to do version 2.0 of Rocket-Nginx. You don't need to work on this issue as I'm 90% done.

If I can count on you to test it when I release it, that would be a better investment of your time!

Thank you for your support.

@CreaoticX
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will do.

Please sign in to comment.