Skip to content

Task download restriction#203

Open
rogi1609 wants to merge 1 commit intodevfrom
task-download-restriction
Open

Task download restriction#203
rogi1609 wants to merge 1 commit intodevfrom
task-download-restriction

Conversation

@rogi1609
Copy link
Contributor

When participating in a challenge, there are several tasks/levels. New tasks can only be shown
respectively downloaded when the previous challenge was submitted and produced the
correct results.
Previously, this was just handled on the client side and users could have downloaded the task
description of the next level by calling the corresponding API. This is not possible anymore as
the permission to download the file is now checked server-side.
This is done by setting a session attribute named "CURRENT_CHALLENGE" which is set when
the challenge is viewed and allows the server to quickly check in what level the user is.

@lorenzleutgeb
Copy link
Contributor

Tests are failing, please fix.

@Exordian do you agree with storing the current challenge inside the session like introduced here?

When participating in a challenge, there are several tasks/levels. New tasks can only be shown
respectively downloaded when the previous challenge was submitted and produced the
correct results.
Previously, this was just handled on the client side and users could have downloaded the task
description of the next level by calling the corresponding API. This is not possible anymore as
the permission to download the file is now checked server-side.
This is done by setting a session attribute named "CURRENT_CHALLENGE" which is set when
the challenge is viewed and allows the server to quickly check in what level the user is.
@rogi1609 rogi1609 force-pushed the task-download-restriction branch from 4966d43 to 556f41a Compare July 29, 2016 07:08
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

Successfully merging this pull request may close these issues.

2 participants