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

[fix] Make signal handler optional #390

Closed
wants to merge 217 commits into from
Closed

[fix] Make signal handler optional #390

wants to merge 217 commits into from

Conversation

NRizzoInc
Copy link

@NRizzoInc NRizzoInc commented Dec 19, 2020

  • closes Allow to set signals #251
  • crow app has a built in signal handler which may interfere with another signal handler
  • solution: add ability to remove crow's handlers, and to add them back
  • credit: @ilejn (I just put up a pr for his suggested patch)

edit:

to remove: app.signal_clear().port(<port>).multithreaded().run()
to add back: app.signal_add(int signal_number)

mrozigor and others added 30 commits August 30, 2018 17:51
Update websocket.h function "void do_read()"
using with instead of open and close
removed while loop
using one line to write the new file
using unordered map instead of ordered map
added indentation to the generated file
…le command would just hang until timeout without fully sending the file).

Added 404 and fixed some issues with windows error message
sendfile still incompatible with windows due to the stat function
also changed the buffer size to 16KB
Test cannot actually test whether the file was sent or not since the test has no access to the socket itself (also idk how to get data that was passed through a socket)
Serving static files using read() and asio::write()
@The-EDev
Copy link

Hi, First off thanks for bringing this up. But unfortunately this repo hasn't been maintained in almost 3 years.

A few people and I are maintaining a separate version here.

Feel free to open any issues or make this same PR there. However I'll still implement this and run the tests and everything if you choose not to.

@NRizzoInc
Copy link
Author

Oh thanks for letting me know! I'll try making a pr over there instead

@NRizzoInc
Copy link
Author

See CrowCpp/Crow#85

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.

Allow to set signals
8 participants