aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Remove -it options from docker execGravatar MarkKoz2019-06-22-3/+3
|
* Add missing steps keyGravatar MarkKoz2019-06-22-0/+1
|
* Run linting & tests in a containerGravatar MarkKoz2019-06-22-14/+28
|
* Fix syntax errors in azure-pipelines.ymlGravatar MarkKoz2019-06-22-6/+7
| | | | | * Add succeededOrFailed condition to code coverage publish task * Rename test publish task for unit tests
* Add CI steps for running & publishing testsGravatar MarkKoz2019-06-22-17/+49
|
* Fix venv being included in coverage when testing in a containerGravatar MarkKoz2019-06-22-4/+7
| | | | * Exclude DEBUG checks from coverage reports
* Add a NsJail log parser testGravatar MarkKoz2019-06-22-3/+33
| | | | | * Add support for debug level to log regex * Change type annotation of log_parse to Iterable
* Rewrite NsJail testsGravatar MarkKoz2019-06-22-58/+80
| | | | | | * Fix SIGSEGV test * Add embedded null byte test * Return None for stderr when there's a ValueError
* Capture NsJail logs from stdout if NsJail fails to parse cmd argsGravatar MarkKoz2019-06-22-3/+9
| | | | Fixes #30
* Use IDs for user and group for consistency across distrosGravatar MarkKoz2019-06-22-2/+2
|
* Fix coverage not finding sourcesGravatar MarkKoz2019-06-22-8/+17
| | | | | | * Mount volume to the same path as the source directory on the host * Keep the container up in the background so it doesn't have to be restarted or the ownership fix
* Fix ownership of coverage fileGravatar MarkKoz2019-06-22-1/+10
| | | | | | When coverage runs in a container, it is ran under root so the resulting coverage file is owned by root. chown is used to change ownership to be the same as the folder it is in.
* Use dev.sh script for running tests and coverageGravatar MarkKoz2019-06-22-94/+20
| | | | * Use unittest and directly use coverage.py instead of pytest
* Create shell script for building a dev image and running a shellGravatar MarkKoz2019-06-22-23/+46
| | | | * Put scripts in a new scripts folder
* Add NsJail alias and switch to ashGravatar MarkKoz2019-06-22-1/+28
|
* Add support for development to Docker imagesGravatar MarkKoz2019-06-22-7/+34
| | | | | | | * Venv image can sync dev dependencies * Copy tests to image * Add a Pipenv script for running a development shell in a container * Add Pipenv scripts for building dev images
* Use a custom Gunicorn access log formatGravatar MarkKoz2019-06-15-4/+18
| | | | | | | | The handler now shares formats with the rest of the handlers. The message is formatted to show the request method, URL + query, response code, response size in bytes, and the request time in seconds. * Use the default ISO 8601 date format for all handlers
* Use system site instead of relying on virtual environment activationGravatar MarkKoz2019-06-15-14/+4
| | | | | | | * Remove -S option from Python to re-enable importing of site module * Add environment variable NSJAIL_PATH * Remove environment variables that were passed to NsJail subprocess * Add type annotations to NsJail.__init__()
* Merge stdout and stderrGravatar MarkKoz2019-06-15-5/+8
| | | | | | Removes the need for redirecting stderr using contextlib in the input. Furthermore, it captures errors which don't directly come from the input, such as SyntaxErrors.
* Merge pull request #24 from python-discord/refactor/nsjailGravatar Leon Sandøy2019-06-09-148/+254
|\ | | | | Improve NsJail
| * Respond to eval with stdout, stderr, and the return codeGravatar MarkKoz2019-06-05-32/+37
| | | | | | | | | | | | | | | | | | The previous implementation limited the client's flexibility in presenting the results of the process. A process can write to both stdout and stderr and do so even when the return code is not 0 or 1. * Return a CompletedProcess from NsJail * Don't check the return code; this should be done client-side now
| * Revert "Add basic logger for responses"Gravatar MarkKoz2019-06-05-16/+1
| | | | | | | | | | Gunicorn's access logger makes the middleware redundant. This reverts commit c1c854e0ad4deb5c4fa8eb2a35b43a042f15255c.
| * Add logging for NsJailGravatar MarkKoz2019-06-05-28/+90
| | | | | | | | | | | | | | | | | | | | | | NsJail's is configured to log to a temporary file rather than stderr. The contents of the file are parsed using regex after the process exits. When not debugging, some blacklisted messages and most info-level messages are skipped. * Add a snekbox logger * Log the Python code being executed if debugging * Use nested single quotes in a test to fix a linter error
| * Merge remote-tracking branch 'origin/revitalisation' into refactor/nsjailGravatar MarkKoz2019-06-05-136/+257
| |\ | |/ |/|
* | Merge pull request #23 from python-discord/falconGravatar Leon Sandøy2019-06-02-163/+278
|\ \ | | | | | | Replace Flask with Falcon
| * | Revise docstrings for SnekAPI & EvalResourceGravatar MarkKoz2019-05-29-9/+9
| | |
| * | Add basic logger for responsesGravatar MarkKoz2019-05-29-1/+16
| | |
| * | Document the APIGravatar MarkKoz2019-05-29-0/+50
| | |
| * | Move SnekAPI import back to top of moduleGravatar MarkKoz2019-05-29-1/+2
| | |
| * | Enable branch coverage & disable app.py coverageGravatar MarkKoz2019-05-29-9/+5
| | |
| * | Add API tests for eval resourceGravatar MarkKoz2019-05-29-1/+65
| | |
| * | Separate application instance from definitionGravatar MarkKoz2019-05-29-6/+16
| | | | | | | | | | | | | | | This way, it is easier for tests to use the application because they can instead create their own instances after patching.
| * | Validate request's inputGravatar MarkKoz2019-05-28-2/+46
| | |
| * | Create falcon APIGravatar MarkKoz2019-05-28-59/+36
| | |
| * | Replace flask dependency with falconGravatar MarkKoz2019-05-28-99/+57
|/ /
| * Configure gunicorn loggingGravatar MarkKoz2019-06-04-10/+24
| |
| * Lint testsGravatar MarkKoz2019-05-30-6/+4
| | | | | | | | | | * Ignore D1xx for tests because they shouldn't require docstrings * Update pre-commit-hooks repo to v2.2.3
| * Revert "Run flake8 via pre-commit for the pipenv lint script"Gravatar MarkKoz2019-05-30-7/+1
| | | | | | | | This reverts commit c2416eb672afc80476fe2da1b364af5d018a16b0.
| * Revert "Lint tests"Gravatar MarkKoz2019-05-30-30/+1
| | | | | | | | This reverts commit e75c764f693c3688a59af0d679e0d3e94f003503.
| * Refactor NsJailGravatar MarkKoz2019-05-30-67/+74
| |
| * Run flake8 via pre-commit for the pipenv lint scriptGravatar MarkKoz2019-05-30-1/+7
| | | | | | | | | | | | | | | | | | Pre-commit is a nice wrapper for this particularily because it allows both the snekbox and tests to be linted with a single command while clearly distinguishing their results. The Python script is a cross-platform implementation of the following: git ls-files -- '*.py' | xargs pre-commit run flake8 --files
| * Lint testsGravatar MarkKoz2019-05-30-1/+30
| | | | | | | | | | Tests ignore all D1xx warnings because tests shouldn't require docstrings.
| * Fix pre-commit configurationGravatar MarkKoz2019-05-30-1/+14
| | | | | | | | | | | | | | | | | | | | | | Excluded files and directories in the flake8 config file are overwritten by the --files argument, which is considered intended behaviour. However, this is problematic due to pre-commit's reliance on the --files argument. Therefore, pre-commit's own exclude config option must be leveraged. * Properly exclude tests directory * Add flake8 plugins as dependencies so that they are actually used
| * Add flake8 plugin to only allow double quotesGravatar MarkKoz2019-05-29-103/+119
|/
* Merge branch 'master' into revitalisationGravatar MarkKoz2019-05-28-9/+58
|\
| * Merge pull request #21 from python-discord/docstring-lint-changeGravatar Chris Goes2019-04-29-9/+1
| |\ | | | | | | Docstring lint change
| | * Remove space between ignored flake8 rulesGravatar Mark2019-04-24-1/+1
| | |
| | * Put back D400 as ignored codeGravatar sco12019-04-24-1/+1
| | | | | | | | | | | | Oops
| | * Relint Snekbox with new linting rulesGravatar sco12019-04-24-8/+0
| | |
| | * Remove docstring newline linting from ignored codesGravatar sco12019-04-24-2/+2
| | | | | | | | | | | | | | | | | | Remove docstring-ends-with-period enforcement to allow for freedom of punctuation. Removes D202, D204, D400
generated by cgit v1.2.3 (git 2.39.1) at 2025-11-04 01:02:23 +0000