search

fluree / core

Fluree releases and public bug reports
0 stars 0 forks source link

Replace @id with https://ns.flur.ee/ledger#ledger in transactions #32

Open cap10morgan opened 9 months ago

cap10morgan commented 9 months ago

@id isn't quite semantically correct as the named graph in question is a transaction, not the ledger itself. It also breaks credential verification because the json-ld canonization code assumes @id values are always IRIs, but our ledger aliases generally aren't.

cap10morgan commented 9 months ago

db PR: https://github.com/fluree/db/pull/578

cap10morgan commented 9 months ago

http-api-gateway PR: https://github.com/fluree/http-api-gateway/pull/72

cap10morgan commented 9 months ago

server PR: https://github.com/fluree/server/pull/3