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

Deprecate & remove additional storage drivers #39

Open
lus opened this issue Jan 30, 2022 · 2 comments · May be fixed by #48
Open

Deprecate & remove additional storage drivers #39

lus opened this issue Jan 30, 2022 · 2 comments · May be fixed by #48
Assignees

Comments

@lus
Copy link
Owner

lus commented Jan 30, 2022

I think removing all storage drivers except the PostgreSQL one would be a good idea.

One major problem when it comes to the implementation of new features that somewhat affect the paste models is that we always have to keep in mind that old data has to be migrateable.

Only the PostgreSQL driver implements auto-migration and maintaining migration logic for every single storage driver would dramatically decrease the development effectivity.
Migrating pastes stored with another driver always include the manual import and export tool which is annoying.

However, we should keep the file storage driver but restrict it to only be accessible when in development mode to not force the developer to set up a fully-fledged PSQL database just to improve some part of the frontend.

Setting up PostgreSQL should not be an issue and I think removing support for other storage options is an acceptable trade-off to achieve a much better development effectivity and flexibility.

@Fastidious
Copy link

This could certainly kill pasty for me (unless I run it always in debug mode). I do not use SQL, just the default file system storage.

@lus
Copy link
Owner Author

lus commented Jan 30, 2022

I think I can agree on only supporting PostgreSQL and SQLite as I can write similar migrations for both of them.
SQLite would allow you to run pasty without setting up a dedicated PostgreSQL server, just like with the current file system storage.

@lus lus linked a pull request Aug 22, 2022 that will close this issue
11 tasks
@lus lus linked a pull request Aug 29, 2022 that will close this issue
11 tasks
@lus lus self-assigned this Jun 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants