search

lblod / app-lblod-harvester

Harvesting Self Service
MIT License
1 stars 4 forks source link

Feature/login #29

Closed aatauil closed 1 year ago

aatauil commented 1 year ago

---- DO NOT MERGE -----

This PR adds the login service to the stack. Together with the frontend-harvesting-self-service it allows to shield data harvested from an authenticated route.

To setup test you can follow the new mu-auth guide in de README (this is part of this PR) and test it together with the frontend PR: https://github.com/lblod/frontend-harvesting-self-service/pull/20

benjay10 commented 1 year ago

(Sorry, I was trying to find a better branch to merge in so that commit from me in another branch can be ignored. No luck.)

I was able to look at this PR and the authentication PR on the frontend and everything seemed to work fine for me, given that you create a new user via a query and set the MU_APPLICATION_SALT for the login-service. Maybe you could add this and an example query to the README for testing purposes?