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

Switch REST library from served to Crow #43

Open
wants to merge 2 commits into
base: feature/spdlog-logging
Choose a base branch
from

Conversation

ClemensElflein
Copy link
Member

Since served development was halted (https://github.com/meltwater/served) and the CORS requests were broken for newer browsers, I decided to swap to Crow (https://crowcpp.org/master/) as REST library in this PR.

@ClemensElflein ClemensElflein requested a review from maehw December 20, 2024 21:24
@maehw
Copy link
Collaborator

maehw commented Dec 21, 2024

I think this is mixed with other changes (like those from switching logging to sdplog).

The relevant changes are in .gitmodules, CMakeLists.txt, dependencies/crow/CMakeLists.txt, dependencies/served and especially in src/api/BoothApi.h/cpp, right?

Edit: I think you have gotten rid of the only git submodule, so the --recursive hint can be removed from README.md as well unless I am missing something.

@ClemensElflein ClemensElflein changed the base branch from master to feature/spdlog-logging December 21, 2024 21:15
@ClemensElflein
Copy link
Member Author

Yes, this builds on the other PR so that we don't have conflicts. I have now changed the base branch, so the changelog in github only shows the relevant changes.

Yes, true. That's not necessary any more.

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.

2 participants