rewriteCookiePath option, to rewrite the "Path" from "Set-Cookie" response headers

[gil]

No description provided.

[drewzboto]

Thanks, appreciate the PR. Let me think a bit on how we can make the rewrite configuration a bit nicer (for both the rewrite and cookie). I'd prefer to make it obvious those to aspects are related

[gil]

Cool! I'll try to think about it here and how it can be better implemented.

[gil]

Maybe something like this?

rewrite: [
    new ContextRewrite('^/changingcontext', '/anothercontext'),
    new ContextRewrite('^/api', '/api2')
    new CookiePathRewrite('^/api', '/')
]

[drewzboto]

If we could provide standard functions instead of using a new instance, that would work nicely. That would also allow people to tuck in their own functions that aren't OOTB.

var proxyUtils = require("./lib/utils.js");
...
rewrite: [
proxyUtils.rewriteContext(...),
proxyUtils.rewriteCookie(...),
function () { ... }
]

This would be a breaking change to the rewrite configuration. But probably worth it for the extra flexibility.

[rrarunan]

Is there an update on the rewrite cookie functionality?

[drewzboto]

should have something out over the holidays

[hco]

Any progress on this? :)

[solidstudio]

also interested in this.

[drewzboto]

Sorry, still on my plate. 2014 started off a bit crazy, haven't had much time to dedicate to this project. Should expect things to slow down shortly to give this another kick.

[anton-rodin]

Come on guys, I can't live without this feature. Please Implement cookieRewrite

[gil]

Hi guys, can anyone check my new pull request? I've updated it to the newer version and with the syntax discussed here. I just didn't have the time to do better testing, so please help me test it before accepting the pull request.

You can add to your "package.json":

"grunt-connect-proxy": "git+https://github.com/gil/grunt-connect-proxy.git#rewrites_new"

[gil]

Also, I've changed in a way that it will still support the old syntax, so it won't break current users.

[drewzboto]

Thanks for the update! I'm not too concerned about the old syntax, i think we'll have to go this route for the next major version. let me give this a spin over the next week or so.

[gil]

If you don't want to support the old syntax, just remove these lines: https://github.com/gil/grunt-connect-proxy/blob/10a6ca6e886fdaf4444e7eaff410640e22bf8760/lib/utils.js#L81-L90

I'll try to do better testing during the week. If you need anything, just let me know.

[drewzboto]

Committed as part of #50 in 0.2.x branch. Will track in #50