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

Redirect to URL instead of Route #473

Closed
tad3j opened this issue Jun 22, 2014 · 6 comments
Closed

Redirect to URL instead of Route #473

tad3j opened this issue Jun 22, 2014 · 6 comments

Comments

@tad3j
Copy link

tad3j commented Jun 22, 2014

If I'm not mistaken this used to work but I guess that it was changed?
However, I'm using this feature in combination with ZfcRbac, where they pass URL instead of the route, so a "route not found" error is thrown. I think that having URL redirect is more flexible than route, since it allows redirecting to other domains and/or redirecting with URL parameters), that's why I'm opening this issue here (initially opened for ZfcRbac).
Would you agree and is there any tendency to make those to compliant between each other?

@teseo
Copy link

teseo commented Jun 23, 2014

I believe this is related to this issue #468

@Colmea
Copy link

Colmea commented Jun 27, 2014

Hi,

I don't want to make another issue, so I reply to this one, but why the redirect feature switch to route base URL in this new release ?
It breaks my whole application and I don't really see what's the vulnerability. Is there a way to make this new choice "backward compatible" ?

By the way, thank's for your module :) !

@Danielss89
Copy link
Member

Currently working on implemented a callback feature so that the developer can choose what should happen.

@mtudor
Copy link

mtudor commented Jul 2, 2014

Hi @Danielss89 - thanks for the work you're doing.

I'm curious about how the callback feature will work. Will there be a default behaviour? Will there be a chance for us to review the PR before it gets merged to the master? Would be happy to offer feedback if it would help.

Cheers!

@Danielss89
Copy link
Member

Please take a look at #480

@Danielss89
Copy link
Member

I released v. 1.2.0 of ZfcUser which makes it possible to change the redirect callback. Check #487 for more info.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants