Revert error handling change in KeycloakAdmin.get_group_by_path

[dominik-korsa]

Fixes #675.

The reverted change was introduced in #627.

Please note that just as #627, this is a breaking change, but it might be acceptable, as raising an exception should not make existing code incorrect.

[ryshoooo]

Thanks a lot @dominik-korsa and co for bringing this back up. I believe all your points are valid, especially for consistency across the library, does not make sense for one method not to raise GET exception while others do.

Twice now we have changed the behavior of this function in a breaking manner without actually doing a major version patch. Which sucks. I do not feel like doing it for the 3rd time. I'm happy with the changes here, but I'd like to release this as a breaking change. I do understand that this is technically not breaking as it has been released already, but those releases were unforeseen with a breaking change. Had I spotted the breakage, it'd have been released as such.

Would that be agreeable?

[dominik-korsa]

Sure, I also think bumping the major version with this change is the correct way to go. Thanks for the detailed response btw!