Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

additional decoding fix #300

Merged
merged 1 commit into from Jun 7, 2012

Conversation

Projects
None yet
2 participants
Contributor

qnikst commented Jun 7, 2012

there is a problem that getPage tries to decode URL when it was already decoded by ackRq, so non-ASCI urls fails to work. This patch fixes situation.

Owner

jgm commented Jun 7, 2012

Looking over my earlier commits, I see:
a61a83d
and then:
10521a8

I wish I remembered why I reverted this commit, which seems the same as your fix. Perhaps my happstack version was not up to date. I'll have to look into this. What version of happstack are you linking with?

@jgm jgm added a commit that referenced this pull request Jun 7, 2012

@jgm jgm Merge pull request #300 from qnikst/master
additional decoding fix
0badae9

@jgm jgm merged commit 0badae9 into jgm:master Jun 7, 2012

Contributor

qnikst commented Jun 7, 2012

I used latest avaliable happstack i.e. 7.0.0 If there will be any problem with old version I can add flag conditional code.

Owner

jgm commented Jun 7, 2012

I just bumped the version bound to >= 7.0.
Thanks for the fix!

+++ Alexander Vershilov [Jun 07 12 11:53 ]:

I used latest avaliable happstack i.e. 7.0.0 If there will be any problem with old version I can add flag conditional code.


Reply to this email directly or view it on GitHub:
#300 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment