search

LogicAppCollaborative / PsLogicAppExtractor

MIT License
3 stars 3 forks source link

Increase JSON max depth from 20 to 100 #34

Closed angularsen closed 2 years ago

angularsen commented 2 years ago

Logic apps are notorious for having deep JSON structures and in our case we had several logic apps with truncated JSON output. A warning was displayed for each.

It is still useful to have a max depth to avoid infinite recursion errors.

Changes

angularsen commented 2 years ago

I'm sure this could be improved by making it configurable somehow, but that is out of my league in terms of understanding psake and this project's setup.

Splaxi commented 2 years ago

Great observation and thanks for taking the time to share the PR.

I'll merge it and let you know when the latest version is available from powershellgallery.

angularsen commented 2 years ago

Wow, quick response! 😃 Thanks!

Splaxi commented 2 years ago

In 10-15 minutes the updated version should be available for you to download.

angularsen commented 2 years ago

Took it for a spin and it worked like a charm, thanks 🙌

Splaxi commented 2 years ago

That is totally your doing - thx for trying out the module and even more for the PR 🤘