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

Add privacy considerations about status message updates/alterations. #156

Merged
merged 4 commits into from
Mar 30, 2024

Conversation

msporny
Copy link
Member

@msporny msporny commented Mar 24, 2024

This PR is an attempt to address issue #146 by documenting how status messages could be used to accidentally expose more information about a holder than they originally thought and how holder software could be used to warn the holder of changes in status message information.


Preview | Diff

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated
For example, an <a>issuer</a> could automatically re-issue a
<a>verifiable credential</a> every three months to break any sort of long-term
monitoring of a <a>verifiable credential</a> as it changes status and assign
a new status entry index when the re-issuance occurs.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
a new status entry index when the re-issuance occurs.
a new status entry index when the reissuance occurs.

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved

<p>
Once a <a>verifier</a> knows of a status list and entry index that is
associated with a specific <a>holder</a>, it becomes possible for that
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
associated with a specific <a>holder</a>, it becomes possible for that
associated with a specific <a>issuee</a>, it becomes possible for that

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that the holder can change but the issuee never changes

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change requires that the issuee term be accepted and defined by the WG. This has been tried and failed several times. It is not appropriate to insert it via every PR where it might be relevant.

It might be appropriate to create another "define issuee" issue and an associated PR in each repo which puts it into every relevant spot in the existing documents, such that the VCWG can evaluate it in place, instead of in theory or in one location or associated spec as was previously (and again here) tried. This would require substantial and ongoing effort, especially in keeping such PRs synced with the live documents in progress, but I don't see any alternative if issuee is to be successfully added to VCDMv2 and associated specs.

All that said — the issuee (whether or not that term is defined and adopted by the WG) is a special case of a holder, and the status list and entry index association here is with a specific holder which may be but is not necessarily the issuee of that VC.

Copy link
Member Author

@msporny msporny Mar 30, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm rejecting the issuee language in this PR until we have consensus to add the term in the VC Data Model. If we have that, then all of the specs will have to be updated to use that base terminology and we'll revisit this change suggestion in a future PR.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this might be more accurate, and pass muster.

Suggested change
associated with a specific <a>holder</a>, it becomes possible for that
associated with a specific <a>holder</a> or <a>subject</a> , it becomes possible for that

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, Thankyou.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this small change needs a new PR to get into the document. I think the other change suggestions have all been applied.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@msporny can you do this please?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See new PR #163

index.html Outdated Show resolved Hide resolved
index.html Outdated
status without asking the <a>issuer</a> directly for status information on
the specific <a>holder</a> or when interacting with the <a>holder</a> to get
the latest status information is not possible. The feature can also cause a
privacy violation if the <a>holder</a> is unaware of the ability for the
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The privacy violation is about the subject isn't it? The sentence implies it's about the holder.
Furthermore the privacy violation may occur regardless of whether the holder is aware or not.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not believe this issue has been resolved if the suggested changes have been ignored, because we currently have no documented mechanism for the VC to contain personal information about the holder, but only about the subject.

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated

<p>
<a>Issuers</a> can provide a level of reprieve from this privacy concern to
<a>holders</a> by revoking and re-issuing effectively the same
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<a>holders</a> by revoking and re-issuing effectively the same
by revoking and re-issuing effectively the same

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The concern is not only to the holder, but could be to the subject and relatives of the subject (e.g. consider that I hold the VC passport of my child, then the privacy violation is of concern to me, my wife and my broader family)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should not discuss any "relatives of the subject" here, unless this is also raised everywhere else it is relevant, which would be a number of locations in several documents. That said, I think the edits (1, 2) that make this about both holder(s) and subject(s) are appropriate.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The point still stands as the text gives the wrong impression if it only talks about the privacy of the holder.

index.html Outdated Show resolved Hide resolved
index.html Outdated
</p>

<p>
This feature creates a potential privacy violation where <a>holder</a> of the
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
This feature creates a potential privacy violation where <a>holder</a> of the
This feature creates a potential privacy violation where the <a>subject</a> of the

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q. Is this status information about the subject or the issuee?
It cannot be about the holder since the holder can be anyone and is not a fixed entity.

index.html Outdated Show resolved Hide resolved
their level of information exposure when using <a>verifiable credentials</a>
that are associated with status messages and warn them when the level of
information exposure changes.
</p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This paragraph is unclear, since it not clear who the status messages are about.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@TallTed clarified the language in some of his change suggestions, which have now been applied. The status messages are about VCs, which express information about subjects/holders.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In which way do VCs express information about holders? (when they are not subjects)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@David-Chadwick

it not clear who the status messages are about

Status messages are always about VCs/VPs; in other words, your question should be "what", not "who". In this sentence, the status messages are about <a>verifiable credentials</a> that are associated with status messages.

In which way do VCs express information about holders? (when they are not subjects)

If nowhere else, VCs express information about holders when issuees are identified, as issuees must be holders. VCs might also express information about authorized presenters who must also be holders, as they cannot present something they don't hold.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Privacy issues can only be about living people. They are not about verifiable credentials, but about the subjects of them (or the issuee if this not-yet-accepted definition is accepted and the property is added to the VC, which no-one is currently proposing). Hence my original question.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@David-Chadwick — I think your original question has been addressed by subsequent changes to the text, which now reads:

Holder software can provide features to <a>holders</a> that warn them about the
level of <a>holder</a> and/or <a>subject</a> information exposure when using
<a>verifiable credentials</a> that are associated with status messages, and warn
them when the level of information exposure changes.

To rephrase, <a>verifiable credentials</a> that are associated with status messages expose some level of <a>holder</a> and/or <a>subject</a> information, about which level, and changes to that level, holder software can warn <a>holders</a>. I think this rephrasing is more confusing than the current text, but perhaps it will clarify things for you?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The text which now reads..... does address my issue. Thankyou. But I cannot understand your re-phrasing paragraph below it. Sorry. I agree with you that it is very confusing!!

index.html Outdated Show resolved Hide resolved

<p>
Once a <a>verifier</a> knows of a status list and entry index that is
associated with a specific <a>holder</a>, it becomes possible for that
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change requires that the issuee term be accepted and defined by the WG. This has been tried and failed several times. It is not appropriate to insert it via every PR where it might be relevant.

It might be appropriate to create another "define issuee" issue and an associated PR in each repo which puts it into every relevant spot in the existing documents, such that the VCWG can evaluate it in place, instead of in theory or in one location or associated spec as was previously (and again here) tried. This would require substantial and ongoing effort, especially in keeping such PRs synced with the live documents in progress, but I don't see any alternative if issuee is to be successfully added to VCDMv2 and associated specs.

All that said — the issuee (whether or not that term is defined and adopted by the WG) is a special case of a holder, and the status list and entry index association here is with a specific holder which may be but is not necessarily the issuee of that VC.

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated

<p>
<a>Issuers</a> can provide a level of reprieve from this privacy concern to
<a>holders</a> by revoking and re-issuing effectively the same
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should not discuss any "relatives of the subject" here, unless this is also raised everywhere else it is relevant, which would be a number of locations in several documents. That said, I think the edits (1, 2) that make this about both holder(s) and subject(s) are appropriate.

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
msporny and others added 2 commits March 30, 2024 14:59
Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
Co-authored-by: David Chadwick <d.w.chadwick@truetrust.co.uk>
Co-authored-by: Dave Longley <dlongley@digitalbazaar.com>
@msporny
Copy link
Member Author

msporny commented Mar 30, 2024

Editorial, multiple reviews, changes requested and made (except for one that was deferred until later), no objections, merging.

@msporny msporny merged commit 837e500 into main Mar 30, 2024
1 of 2 checks passed
@msporny msporny deleted the msporny-status-update-dangers branch March 30, 2024 19:09
Copy link
Contributor

@David-Chadwick David-Chadwick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its difficult to know what the final text currently is as so many changes have been proposed to it

their level of information exposure when using <a>verifiable credentials</a>
that are associated with status messages and warn them when the level of
information exposure changes.
</p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In which way do VCs express information about holders? (when they are not subjects)


<p>
Once a <a>verifier</a> knows of a status list and entry index that is
associated with a specific <a>holder</a>, it becomes possible for that
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then please change holder to subject.

index.html Outdated
status without asking the <a>issuer</a> directly for status information on
the specific <a>holder</a> or when interacting with the <a>holder</a> to get
the latest status information is not possible. The feature can also cause a
privacy violation if the <a>holder</a> is unaware of the ability for the
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not believe this issue has been resolved if the suggested changes have been ignored, because we currently have no documented mechanism for the VC to contain personal information about the holder, but only about the subject.

index.html Outdated

<p>
<a>Issuers</a> can provide a level of reprieve from this privacy concern to
<a>holders</a> by revoking and re-issuing effectively the same
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The point still stands as the text gives the wrong impression if it only talks about the privacy of the holder.

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

Successfully merging this pull request may close these issues.

6 participants