On Friday, June 5, 2020 at 10:57:51 PM UTC+2, Maciej Sobczak wrote:
> Z przyjemnością informuję o narodzinach wersji 1.0 nowego pakietu:
>
> http://www.inspirel.com/httpserver/
>
> Jest to bardzo prosta biblioteka w C++11, dla systemów Linux, Mac OS X oraz
Windows, pozwalająca progamowi w C++ wyświetlić swoje GUI na przeglądarce webowej.
>
> Idea jest również opisana tutaj:
>
> https://en.wikipedia.org/wiki/Embedded_HTTP_server
>
> i wbrew nazwie nie ma bezpośredniego związku z systemami wbudowanymi (chociaż na
RaspberryPi i podobnych można by było z satysfakcją domknąć tą definicję).
>
> Wszelkie komentarze mile widziane.

Po pierwsze, to aż się prosi, żeby ten serwer był zwykłą klasą. Czemu ktoś nie miałby
sobie stworzyć 5 serwisów WWW działających na różnych portach?

Po drugie, nie ma sposobu na zamknięcie serwera, poza zabiciem procesu. Chyba, że ja
czegoś nie widzę.

Mówiąc o API: 6 wariantów register_{generic,html,text}_{post,get}_action można by
sprowadzić do 2. register_action(const char* name, {post_action_type,
get_action_type} callback, const char* mime_type). Zresztą, może lepiej byłoby
przyjmować jako argument mapę akcji - czyli to co masz teraz w
get_actions/post_actions. Niech user sobie przygotuje takie mapy w sposób, jaki mu
pasuje, a ty pozbędziesz się potrzeby blokowania tych struktur.

Czasem funkcje przyjmują const char*, czase std::string& co jest niespójne.

W ogóle nie walidujesz, czy wskaźniki są niepuste. Tak samo z std::function.

Zero testów. Serio? :)

w.