Skip to content
This repository was archived by the owner on Dec 30, 2022. It is now read-only.

Reformat and organize, more efficient checking, stats...#2

Open
olvrb wants to merge 11 commits into
Sileo:masterfrom
olvrb:master
Open

Reformat and organize, more efficient checking, stats...#2
olvrb wants to merge 11 commits into
Sileo:masterfrom
olvrb:master

Conversation

@olvrb

@olvrb olvrb commented Dec 30, 2018

Copy link
Copy Markdown

Changes

  • Reorganize all routes into a separate folder: controllers/.
  • Set up a database system (Sequelize with pg) to store statistics (note: no personal data to identify a user).
  • In addition to the last point, create a new config file (config/config.js), readme has been updated accordingly.

@nullpixel

Copy link
Copy Markdown
Contributor

Looks good, but did you remove the regex stuff?

@nullpixel

Copy link
Copy Markdown
Contributor

“Simplify the way we check if a request is a piracy repo“ — this was intentionally complex so we can wildcard domains and paths and do other useful stuff

@olvrb

olvrb commented Dec 30, 2018

Copy link
Copy Markdown
Author

Ah, I see. I'll revert it back to the previous regex stuff.

@olvrb

olvrb commented Dec 31, 2018

Copy link
Copy Markdown
Author

The changes have been reverted, and the database changed to SQLite for simpler setup.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants