Enable policy opts in query and transact #46
Conversation
| @@ -4,3 +4,4 @@ | |||
| /target/* | |||
| .clj-kondo/babashka | |||
| .clj-kondo/metosin | |||
| .dir-locals.el | |||
There was a problem hiding this comment.
This is the tiniest of nitpicks but these editor-specific ignores might be better in your global gitignore (e.g. ~/.gitignore which I think you have to then tell git to use in its config). However, I know Emacs is a pretty popular choice around here and for Clojure in general 😉 so I'm fine with it either way.
There was a problem hiding this comment.
We have this in our gitignores on db and json-ld, so I actually originally assumed its absence here was an oversight.
Emacs is pretty popular among lispers, so it's convenient to have it, but I won't die on this hill either 😄
test/fluree/http_api/system_test.clj
Outdated
| {:ledger ledger-name | ||
| :query (assoc secret-query | ||
| :opts {"role" "ex:userRole" | ||
| "did" alice-did})}) |
test/fluree/http_api/system_test.clj
Outdated
| (-> query-res :body json/read-value)) | ||
| "query policy opts should prevent seeing bob's secret") | ||
| (let [txn-req {:body | ||
| (json/write-value-as-string |
|
@cap10morgan thanks for reminding me about edn, I added support and some tests. And I had actually neglected to clean up my whitespace generally, so I just ran some formatting on all my new tests and pushed it up. It ended up changing more than what you had suggested, but it should now be consistent at least. |
Closes #38
Depends on fluree/db#445
Adds handling of policy-relevant opts (
"did"and"role"for now), so that they can be passed in to queries and transactions.