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

Countries not supported return 204 response by IsTodayPublicHoliday #404

Open
CookieMonk opened this issue Jul 13, 2022 · 2 comments
Open

Comments

@CookieMonk
Copy link

More information

Hi Guys,
First of all, thanks for creating this project!

I was testing the IsTodayPublicHoliday request using various countries: PL, GB, GR and all works fine.
However, I've started playing with countries that are not supported, like Indie - IN, Ethopia - ET and instead of getting 404 response, I got 204, which was a bit confusing.

Right now I need to first find the country on the list of supported countries (AvailableCountries) and if there is a match run IsTodayPublicHoliday.

Is this a "feature" or a bug :)

Regards,
Marcin

@tinohager
Copy link
Member

This is because I can't get along with the 404 code, it gives me too much of a feeling that the endpoint doesn't exist. But with a 400 I could make friends.

@CookieMonk
Copy link
Author

Thanks for the reply!
400 "Validation Error" Sounds good for me. Also country code that contain 3 letters or 1 letter, both of course provided by mistake, also return 404, so after the change it would be good to get 400 too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants