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

Improve security within the frontend and the backend #42

Closed
mytja opened this issue Jul 11, 2022 · 2 comments
Closed

Improve security within the frontend and the backend #42

mytja opened this issue Jul 11, 2022 · 2 comments
Assignees
Labels
Milestone

Comments

@mytja
Copy link
Member Author

mytja commented Sep 13, 2022

Currently working on it and it will contain major & breaking changes.

All tables, except user table in SQL will need to be dropped (using the provided SQL migration), as that's the easiest way of doing it. As no school is presently using the system, I can allow these kinds of migrations with dropping tables, but I PROMISE, that once schools actually install the system, I will do everything (if I can't do anything, then I will revert all my local changes to latest stable and won't implement the feature) in order to KEEP THE DATA INTACT during the migrations.

What's new, that you need to drop tables?

  • Move away from auto-increment IDs, to UUIDs

This will help us improve code quality, as we'd be able to generate UUIDs directly in SQL. This will also help user privacy, as you can't identify other people (how many users there are) just by using the ID.

Of course, this change will be firstly pushed into beta branch. NO CURRENT INSTANCES will be affected until a manual code review. After a manual code review and extensive testing, this will be merged into main branch. THE MERGE WILL BE DELAYED BY ONE DAY, for anybody to maybe opt-out of the auto-update system. You'll find all the details about it within the MeetPlan Documentation.

Again, this is for the best, as security is massively improved and by not updating, you risk of having potentially useful information (such as amount of users in your system), leaked to the public.

Thank you so much for understanding
Mitja Ševerkar
The MeetPlan Team

@mytja mytja added this to the v2.1 milestone Sep 13, 2022
@mytja
Copy link
Member Author

mytja commented Feb 14, 2023

Oh, so this has been merged I see, closing now.

@mytja mytja closed this as completed Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

1 participant