Skip to content

Conversation

@aleksen
Copy link

@aleksen aleksen commented Feb 12, 2026

Pull Request

Description

When I was trying to run marchat server locally to test it out, I used the interactive setup guide. It did not ask me for the JWT Secret, so it was set to a hardcoded string. Perhaps this could be a problem for other people setting up marchat servers using the interactive flag. So I could either add the JWT Secret to the interactive setup, or make the default be a generated hex value.

Also, some of the tests failed on my computer, since I already had a working setup in ~/.config/marchat. I inserted t.TempDir() in the tests that used the default location.

Fixes # (issue)

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Checklist

  • My code builds and passes all tests/CI
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated documentation as needed
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • I have assigned reviewers

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant