add deprecation warnings for redirect arguments undefined

Open bjohansebas opened this issue 9 months ago • 3 comments

:stop_sign: No Merge before #6404 has landed in v6

With this, those arguments are deprecated when they are undefined or different from their data type

Related #6404 Closes #6391

Mar 19 '25 00:03 bjohansebas

I added a warning message in the description to prevent early merges :+1:

Mar 19 '25 12:03 UlisesGascon

I think it should be the other way around—this should be merged first before #6404, since it needs to be deprecated in v5 first for the effect to result in throwing errors in v6. Does that make sense?

Mar 20 '25 21:03 bjohansebas

@wesleytodd @UlisesGascon @expressjs/express-collaborators Any objections to releasing this in the next minor version of Express?

May 29 '25 00:05 bjohansebas