(maint) Bump http-client dependency

This fixes an annoying warning about get being overridden
(NC-157) Error when a group is its own parent
(NC-157) Detect inheritance cycles larger than one group
(NC-157) Detect cycles in get-ancestors
(NC-157) Make sure cycles can be fixed

Test that you can break a cycle by altering the parent_id of a group in the
cycle. This required making get-subtree* not follow cycles involving the root
of the subtree. It is not protected from cycles further down the subtree.
(NC-157) Include offending cycle in the inheritance-cycle exceptions
(NC-157) Return a nicer error in the JSON for cycles
(NC-157) Document error response for group inheritance cycles
(NC-157) Refactor get-ancestors

Clean up the conditionals in get-ancestors a bit
(NC-157) Clarify and refactor code around get-subtree

This refactors get-subtree slightly, refactors the tests around it, and adds
some clarifying documentation.
(NC-157) Use name instead of UUID for inheritance-cycle message

Since the full groups are included in the details key, using group names
instead of the UUIDs in the plain english error will probably be more user
friendly without introducing too much ambiguity.
Merge pull request #60 from pcarlisle/nc-157-strict-hierarchy

(NC-157) Detect inheritance cycles in groups