Success HTTP Status Codes

kutuluk / loglevel-plugin-remote

A loglevel plugin for sending browser logs to a server

MIT License

102 stars 36 forks source link

Success HTTP Status Codes #24

Open james-r-parker opened 4 years ago

james-r-parker commented 4 years ago

It appears that you are only checking for a HTTP status code of 200 when getting a response back from sending log entries to the server. This check should also check the entire 200 range for example 201 and 202 and 204

cchapman-tlcdeaf commented 4 years ago

I was having this same issue. My API returned 201 instead of 200, and I would get continuous logs being sent. It seems like it should be an easy update.

sergeicodes commented 4 years ago

Same issue for me, 201 is returned. Any plans to fix it?

emrekorus commented 2 years ago

Any Update?

ghost commented 2 years ago

I created a MR for this. https://github.com/kutuluk/loglevel-plugin-remote/pull/43 as it was quite annoying for me. Personally i started to create a local copy and modify it for my own purposes.