search

MajidBenam / seshat

This is the beginning of the Seshat Project code.
0 stars 3 forks source link

Removing exposed user data #13

Closed kallewesterling closed 1 month ago

kallewesterling commented 1 month ago

Currently, the website's API exposes all the users of the platform (and looks like it's done so for ~2 years time).

See https://seshat-db.com/api/users/ for the exposed end point.

Please merge this PR as soon as possible to stop the API route from existing, as a first step.

MajidBenam commented 1 month ago

Thank you so much. I took care of it.

On Thu, Jun 6, 2024 at 7:02 PM Kalle Westerling @.***> wrote:

Currently, the website's API exposes all the users of the platform (and looks like it's done so for ~2 years time).

Please merge this PR as soon as possible to stop the API route from existing, as a first step.

You can view, comment on, or merge this pull request online at:

https://github.com/MajidBenam/seshat/pull/13 Commit Summary

File Changes

(1 file https://github.com/MajidBenam/seshat/pull/13/files)

Patch Links:

— Reply to this email directly, view it on GitHub https://github.com/MajidBenam/seshat/pull/13, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARAM4SGZXQITQBKZ4QCIN7LZGCIZHAVCNFSM6AAAAABI5EBFWSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGMZTQNZVGUYDENI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

kallewesterling commented 1 month ago

Closing as @MajidBenam included the edit in his 7aa32a6 commit.