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

[Implementation Guide] - JSON Path deps #419

Open
2 tasks
bumblefudge opened this issue Mar 2, 2023 · 5 comments
Open
2 tasks

[Implementation Guide] - JSON Path deps #419

bumblefudge opened this issue Mar 2, 2023 · 5 comments
Assignees
Milestone

Comments

@bumblefudge
Copy link
Collaborator

bumblefudge commented Mar 2, 2023

  • colby pointed out today that the recommended kotlin lib removes "in-path functions" -
    • is that a problem? will that make holders throw when interacting with an eval-happy verifier? ongoing discussion about how implementations with very-liberal and very-locked-down security policies re: filter expressions can gracefully interact without just dropping or throwing
  • decision taken on this week's call to figure out some reasonable guidance on constraining/sandboxing JSONPath and auditing recommended libraries against that guidance before putting back in
  • decision taken on this week's call to remove impl recommendations and [carefully] put back in to future implementation guide
@colbyharrison
Copy link

I would broaden this question to the whole Developer Resources section

@bumblefudge
Copy link
Collaborator Author

edited original message and issue name according 💪

@kimdhamilton kimdhamilton added help wanted Extra attention is needed ready for pr Next steps known; ready for PR and removed help wanted Extra attention is needed labels Apr 6, 2023
@brentzundel brentzundel removed the ready for pr Next steps known; ready for PR label Apr 6, 2023
@bumblefudge
Copy link
Collaborator Author

bumblefudge commented Jun 15, 2023

Discussed on today's call:

@bumblefudge
Copy link
Collaborator Author

@rado0x54 how can we help you, ser

@rado0x54
Copy link
Contributor

rado0x54 commented Oct 5, 2023

@bumblefudge Terribly sorry ser, I thought this issue was done, since ALL implementation recommendations have been removed in #438. So good to close?

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

No branches or pull requests

5 participants