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

Make Bee config break by default #24

Open
j2ein opened this issue Sep 30, 2024 · 1 comment
Open

Make Bee config break by default #24

j2ein opened this issue Sep 30, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@j2ein
Copy link
Collaborator

j2ein commented Sep 30, 2024

Is your feature request related to a problem? Please describe.
Currently, the bee config is written in a way that the default config without a valid registration token is valid and can be used.

Describe the solution you'd like
It would be better to write it in a way that it breaks, or give a meaningful error message when the registration token is the default.

Describe alternatives you've considered
None.

@j2ein j2ein added the enhancement New feature or request label Sep 30, 2024
@felixauringer
Copy link
Collaborator

In #25, I added a new log message to make it easier to find the problem. However, I would argue that the real problem is the Internal Server Error that the API returns if the token is not valid. It should return a more meaningful error so that we can be sure (and log) that the problem is indeed the token.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants