EngageSoftware / Engage-Employment

Engage: Employment is a job listing and applicant management tool for the DNN Platform.
7 stars 9 forks source link

Always use absolute urls with domain #5

Closed vmasanas closed 9 years ago

vmasanas commented 9 years ago

Fix for exported urls

bdukes commented 9 years ago

Thanks Vicenç :smile:

I did notice that if you're logged in, there's an issue where the initial request doesn't have your credentials on it, but then when you're redirected to the login it sees that you are logged in, and redirects you to the home page. But that's more of a DNN bug than anything directly with Employment.

bdukes commented 9 years ago

I've published it as 1.12.7 on the forge

vmasanas commented 9 years ago

Brian, actually that's something I've also noticed and can't undertand. If I click the link from excel I get to the login page always, even when I clearly see I'm logged in as host. Instead, if I copy/paste the same url from excel into a new browser window I get the document without a problem.

bdukes commented 9 years ago

Watching the request in Fiddler, the initial request has an MS Office user agent and doesn't pass the auth cookie. I'm not exactly sure at what point that gets passed to the browser though.

Sent from my Windows Phone

-----Original Message----- From: "Vicenç Masanas" notifications@github.com Sent: ‎12/‎20/‎2014 2:19 AM To: "EngageSoftware/Engage-Employment" Engage-Employment@noreply.github.com Cc: "Brian Dukes" dukes.brian@gmail.com Subject: Re: [Engage-Employment] Always use absolute urls with domain (#5)

Brian, actually that's something I've also noticed and can't undertand. If I click the link from excel I get to the login page always, even when I clearly see I'm logged in as host. Instead, if I copy/paste the same url from excel into a new browser window I get the document without a problem. — Reply to this email directly or view it on GitHub.=