Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

St 138 security #201

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

St 138 security #201

wants to merge 11 commits into from

Conversation

alexdany657
Copy link
Member

Please, read the code before running anything.

Script setup adds new user, removes it's password and blocks changing for a long time.
This script must be executed with root privileges.

Also I've added memory limit 256 MB. It's probably a good idea to rename verdict TL(Time limit) to RL(Resource limit). Also may be we want to make memory limit adjustable.

Now we need to add to wiki that admin needs to copy classes Turn and GameState to separate file named classes. May be there is a better solution but this was easier and faster. See tic_tac_toe_exp/sources/classes.py for sample.

@alexdany657 alexdany657 requested a review from cookiedoth August 27, 2019 06:36
@cookiedoth
Copy link
Member

Привет, я завтра посмотрю, ок?

@alexdany657
Copy link
Member Author

Ок

@cookiedoth
Copy link
Member

This script will create user test (if not exist) remove it's password and block password changing for a long time and make some changes in test's home catalog, so this script must be executed with root privileges.
Before running this script make sure that you fully understand what it will do by reading it's code. Author of this script is not responsible for any consequenses.

Хех

@cookiedoth
Copy link
Member

Скачиваю виртуалку......

Вообще, мне кажется, что надо сделать, чтобы можно было локально запускать, без добавления нового юзера (как раньше).

@alexdany657
Copy link
Member Author

Наверное, да, но мне кажется, это лучше вынести в другое issue.

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