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.

Sign up for GitHub

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

Jump to bottom

crypto/verify refactor? #845

Open

bplatz opened this issue Jul 18, 2024 · 1 comment

Contributor

bplatz commented Jul 18, 2024 •

edited

Loading

There is a question if crypto/verify should be doing JSON parsing, or if that should be handled separately.

Once some additional changes in fluree/server land, including how it is used in middleware, it can be examined.

I think at this point the way it is working currently is OK.

bplatz mentioned this issue

Fix sparql query-credential APIs #843

Closed

Contributor Author

bplatz commented Jul 18, 2024

Also to point out - this function is only used internally, so whatever choice is decided it will not affect anything touching end users.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment