-
Couldn't load subscription status.
- Fork 0
Test eService implementation #97
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’ve left some comments, please review them and address each one.
Also, don’t forget to update the main README file of the eID-Server.
You should add a section explaining how to test the server with the test-eService, and include a link to the corresponding folder so others can easily find the documentation on how to set it up and run it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I reviewed the code again, globally it evrything seems ok but I still see some comments that have not been addressed. Could you carefully check them ? Also I don't see any section in the README of the eID-Server about the test setup as mentioned in my previous comment.
You should add a section explaining how to test the server with the test-eService, and include a link to the corresponding folder so others can easily find the documentation on how to set it up and run it.
| ```toml | ||
| your-project/ | ||
| ├── certs/ | ||
| │ ├── server.crt # HTTPS server certificate | ||
| │ ├── server.key # HTTPS server private key | ||
| │ ├── client.crt # mTLS client certificate | ||
| │ ├── client.key # mTLS client private key | ||
| │ └── server-ca.crt # eID-Server CA certificate (optional) | ||
| ├── .env.local | ||
| └── ... | ||
| ``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check the syntax here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.

No description provided.