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.

Sign up for GitHub

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

Jump to bottom

Fix Issues 129 and 130 #133

Merged
merged 2 commits into from
May 10, 2021
Merged

Fix Issues 129 and 130 #133

merged 2 commits into from
May 10, 2021

Conversation

apolopena
Copy link
Owner

  • Fix 130 --> Force nginx trailing slash rewrites to use https rather than $scheme
  • Fix 129 --> Remove the automatic appending of a trailing slash if it was missing.

@apolopena apolopena merged commit e2aa447 into development May 10, 2021
apolopena added a commit that referenced this pull request May 10, 2021
@apolopena
🚀 RELEASE: Version 1.2.0 (#136)
fd611a3 
* 🐛 FIX: #128

* 🐛 FIX: #127 and #128

* 🚀 RELEASE: bump version to 1.1.1

* Bump xdebug to 3.0.4 (#132)

* 🐛 FIX: #129

* 🐛 FIX: #129 missing code

* Fix  Issues 129  and 130 (#133)

* 🐛 FIX: #131

* 🐛 FIX: #130 opening url paths that end in a file name gives a 404

* 🤖 TEST: force Docker image build

* 💡 UPDATE COMMENTS: fix bad wording for include_readme

* 🐛 FIX: #134 and #135

* 🚀 RELEASE: bump gls version to 1.2.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@apolopena