search

xh / hoist-core

🏗️ The XH Hoist toolkit for Grails
https://xh.io/
Apache License 2.0
7 stars 2 forks source link

Update `/xh` endpoints and config for version/instance change detection #387

Closed ghsolomon closed 2 months ago

ghsolomon commented 2 months ago

…ropriate defaults

ghsolomon commented 2 months ago

Associated with HR PR: https://github.com/xh/hoist-react/pull/3768

amcclain commented 2 months ago

Had a discussion with Lee earler this evening and throwing these changes over the wall to see if you guys think this holds together.

While the client-side code does not save down all the data produced by /xh/environment on its polling checks, I was still happy to not need another endpoint and this should always be cheap / safe to call.

Retained /xh/version for compatibility, but delegating to the same. Was also looking at what whitelisted /xh/environment has been returning to unauthenticated clients and felt like it was more than we would actually want - especially the library versions. But do not believe the slimmed down payload would break any current usages.