Opened 17 years ago

Closed 11 years ago

#655 closed enhancement (wontfix)

Does this work with .10dev (svn)

Reported by: vinay Owned by: nik4536
Priority: normal Component: HttpLoginViaRedirectPatch
Severity: normal Keywords:
Cc: vinay@…, jorge.vargas+trac@… Trac Release: 0.10


Since I run a production server, I don't have the ability to take down the system and patch it and see if it works. I would like to know if this works with .10dev. If not, is there a solution that does?

If I do get the chance to test it against a .10dev trac installation, I'll update the ticket here to reflect that.

Attachments (2)

httploginviaredirectpatch-0.10.patch (3.0 KB) - added by James Mills 17 years ago.
Patch for 0.10
auth.diff (1.8 KB) - added by anonymous 16 years ago.
Patch for 0.10.3

Download all attachments as: .zip

Change History (7)

comment:1 Changed 17 years ago by James Mills


I've attached a patch to make this work with 0.10

See: attachment:httploginviaredirectpatch-0.10.patch

Could someone please try to make this also work with the AuthFormPlugin :)

Changed 17 years ago by James Mills

Patch for 0.10

comment:2 Changed 17 years ago by anonymous

I just try out this patch agains 0.10.3 with tracd

I believe the file doesn't needs to be patch anymore with standalone.diff since the split in is taking out the hardcoded comparison agains /login

but for some reason the redirect is not working, here is my tracd output in debug mode

localhost.domain - - [06/Jan/2007 13:55:43] "GET /trac/ HTTP/1.1" 403 -
localhost.domain - - [06/Jan/2007 13:55:59] "GET /trac/login/timeline HTTP/1.1" 401 -
01:55:59 PM Trac[main] WARNING: 404 Not Found (No handler matched request to /login/timeline)
localhost.domain - - [06/Jan/2007 13:55:59] "GET /trac/login/timeline HTTP/1.1" 404 -

as you can see there is no trace of self.env.log.debug("req.path_info = %s" % req.path_info) so I believe it's not getting process correctly.

comment:3 Changed 17 years ago by anonymous

Cc: jorge.vargas+trac@… added

Changed 16 years ago by anonymous

Attachment: auth.diff added

Patch for 0.10.3

comment:4 Changed 16 years ago by anonymous

I patched HttpLoginViaRedirectPatch with attachment:ticket:655:httploginviaredirectpatch-0.10.patch but it does not work with trac 0.10.3. I have attached (attachment:ticket:655:auth.diff) a version of HttpLoginViaRedirectPatch that works.

comment:5 Changed 11 years ago by Ryan J Ollos

Resolution: wontfix
Status: newclosed

Closing since this patch is for 0.9/0.10, and both are long dead now.

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain nik4536.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.