search

topcoder-platform / tc-website

TopCoder website (legacy).
https://coder.topcoder.com/internal/web_module/
2 stars 13 forks source link

Logging in while registering for marathon match drops url param #11

Open ajefts opened 7 years ago

ajefts commented 7 years ago

Try to register for a marathon match if you're not logged in. It's dropping the rd= param. An example URL is https://community.topcoder.com/tc?module=MatchDetails&rd=16996.

After you login you end up at https://community.topcoder.com/tc?module=MatchDetails

ajefts commented 7 years ago

@skyhit think we could run a F2F to figure this out?

skyhit commented 7 years ago

@ajefts are we using this one for marathon match - https://github.com/appirio-tech/arena-longcontest?

ajefts commented 7 years ago

No, I believe it is https://github.com/topcoder-platform/tc-website/tree/dev/src/main/com/topcoder/web/codinginterface/longcontest

I think it's the way it's redirecting to the login. If we get it to accounts.topcoder.com I think we can handle the retUrl correctly without dropping params...

skyhit commented 7 years ago

@ajefts as I know we split before (long time ago), maybe we switch back to use tc-website again.

skyhit commented 7 years ago

@ajefts I tried, but after registration, I was ended up at https://community.topcoder.com/longcontest/?module=ViewActiveContests&rt=13

which is pretty good.

ajefts commented 7 years ago

Thanks for looking. We just figured out a fix so we are ok on this one now.

On Oct 4, 2017, at 9:10 PM, skyhit notifications@github.com wrote:

@ajefts I tried, but after registration, I was ended up at https://community.topcoder.com/longcontest/?module=ViewActiveContests&rt=13

which is pretty good.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.