Skip to content

Switch REST library from served to Crow#43

Open
ClemensElflein wants to merge 2 commits intofeature/spdlog-loggingfrom
feature/switch-REST-library
Open

Switch REST library from served to Crow#43
ClemensElflein wants to merge 2 commits intofeature/spdlog-loggingfrom
feature/switch-REST-library

Conversation

@ClemensElflein
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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