| Commit message (Collapse) | Author | Age | Lines | 
| | 
| 
|  | 
Co-authored-by: Kieran Siek <[email protected]>
 | 
| | 
| 
| 
| 
|  | 
Co-authored-by: Bluenix <[email protected]>
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | 
| 
| 
|  | 
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | 
| 
| 
| 
| 
|  | 
Adds the ability to specify who can edit forms using discord roles.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | 
| 
| 
| 
| 
| 
|  | 
Adds a new property on forms to declare which roles are authorized to
access form responses.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | 
| 
| 
| 
| 
| 
|  | 
Adds discord role support to the pre-existing scopes system to power
more complex access permissions.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | 
| 
| 
|  | 
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | 
| 
| 
| 
| 
|  | 
We currently use WEBHOOK_ENABLED to determine whether user data should be stored to the db. However, when webhooking a form submission this config is ignored, and the user mention is always included if available.
This means that if a user login in using another form, and then submits an anonymous form with the same session, their name will be included in the webhook.
 | 
| | 
| 
| 
|  | 
Note that it will make any existing form with an upper case letter impossible to access until its ID is changed, which shouldn't be the case in production according to @HassanAbouelela
 | 
| |  | 
 | 
| | 
| 
| 
| 
| 
| 
|  | 
Temporarily switches the URL in discord embeds from the frontend, to the
backend, to allow for easier viewing until the frontend is ready.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| |\  
| | 
| |  | 
Allow Unittest Failures
 | 
| | | 
| | 
| | 
| |  | 
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| |  | 
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| | 
| | 
| |  | 
Adds a check to handle code questions with no test suites.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| | 
| |  | 
Adds a comment which explains when a certain return_code is used.
Co-authored-by: Joe Banks <[email protected]>
 | 
| | | 
| | 
| | 
| | 
| |  | 
Uses 422 instead of 403 to indicate a test has failed.
Co-authored-by: Joe Banks <[email protected]>
 | 
| | | 
| | 
| | 
| | 
| | 
| |  | 
Records the reason of failure for non-zero exit codes.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| | 
| | 
| | 
| |  | 
Adds logic for saving submissions of failed unittests. Updates schema
docs.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| | 
| | 
| | 
| |  | 
Adds an explicit model for questions of type `code`, to allow more
complex parsing. Updates schema docs.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| |\ \  
| | | 
| | |  | 
Empty Authorization Form
 | 
| | | | 
| | | 
| | | 
| | | 
| | | 
| | | 
| | |  | 
Returns a form with no questions, if the server is not running in
production, and no other forms are found, to aid in setting up.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | | 
| | | 
| | | 
| | | 
| | | 
| | | 
| | | 
| | |  | 
The form model specifies webhook as nullable, but the validator code
does not properly handle them. This PR adds logic to handle that
scenario.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | |/  
|/|   
| |   
| |   
| |   
| |   
| |    | 
Reverts the changes made in 4f28ae85 because they made it impossible to
send a partial patch request. Changes to a simpler try/except instead.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| |  | 
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| |/  
|   
|   
|   
|   
|    | 
Makes sure patch requests send a validated request.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | 
| 
| 
| 
| 
| 
|  | 
Copies the admin adding endpoint into an unprotected endpoint that is
only registered in non-production builds.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | 
| 
|  | 
Co-authored-by: Hassan Abouelela <[email protected]>
 | 
| |\   | 
 | 
| | | 
| | 
| | 
| | 
| | 
| |  | 
Removes schema from the token cookie's domain field.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| | 
| | 
| | 
| | 
| |  | 
Traefik forwards https traffic to http, which causes issues with the
protocol in a request's URL. This switch uses the protocol header to
correctly set the protocol.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| | 
| | 
| |  | 
Correctly formats the domain set on the cookie used for tokens.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| |  | 
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| |  | 
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| | 
| | 
| |  | 
Adds a helper method to allow tokens to work on deploy previews.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| | 
| | 
| |  | 
This reverts commit 013ea900
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| | 
| | 
| | 
| |  | 
Sets the authorization token cookie's security policy to lax, to allow
it to work on the site.
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| | 
| | 
| | 
| | 
| |  | 
Cleans up the authorize file, and the __init__ to maintain the project's
 code style.
Co-authored-by: Joe Banks <[email protected]>
Signed-off-by: Hassan Abouelela <[email protected]>
 | 
| | | 
| | 
| | 
| | 
| |  | 
Changes the name for the token used to authorize with the backend.
Co-authored-by: Joe Banks <[email protected]>
 | 
| | |\  
| | | 
| | | 
| | | 
| | | 
| | |  | 
# Conflicts:
#	backend/routes/forms/form.py
#	backend/routes/forms/submit.py
 | 
| | | |  | 
 | 
| | | |  | 
 | 
| | | |  | 
 | 
| | | |  | 
 | 
| | | |  | 
 | 
| | | |  | 
 | 
| | | |  | 
 | 
| | | |  | 
 | 
| | | |  | 
 | 
| | | | 
| | | 
| | | 
| | |  | 
Co-authored-by: Hassan Abouelela <[email protected]>
 |