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

Bug fix - Can't purge because of bad url #82

Merged
merged 1 commit into from Jun 11, 2015

Conversation

Projects
None yet
3 participants
@javisperez
Copy link

commented May 9, 2015

The purge url was escaped, so the ampersand character (&) was encoded, converting the query string into a hashtag.

Signed-off-by: Javis Perez javisperez@esd.com.do

Javis Perez
Bug fix - Can't purge because escaped url was encoding the ampersand …
…character (&), converting the query string into a hashtag

Signed-off-by: Javis Perez <javisperez@esd.com.do>
@gsf

This comment has been minimized.

Copy link

commented Jun 10, 2015

Please merge this request. It will fix #89 and address http://community.rtcamp.com/t/nginx-helper-doesnt-purge/4462.

@rahul286 rahul286 merged commit bd9ad41 into rtCamp:master Jun 11, 2015

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/wercker Wercker build passed
Details
@rahul286

This comment has been minimized.

Copy link
Member

commented Jun 11, 2015

Thanks for PR @javisperez. I am sorry for delayed reply.

@gsf

This comment has been minimized.

Copy link

commented Jun 11, 2015

Awesome, thanks. Ready to release 1.8.11?

@rahul286

This comment has been minimized.

Copy link
Member

commented Jun 12, 2015

@gsf
Done. Version is 1.8.12

Please let me know if you face any issues with it.

@gsf

This comment has been minimized.

Copy link

commented Jun 16, 2015

That works! Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.