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

Draft -01 #4

Merged
merged 21 commits into from
Jun 26, 2024
Merged

Draft -01 #4

merged 21 commits into from
Jun 26, 2024

Conversation

xcolwell
Copy link
Collaborator

@xcolwell xcolwell commented May 6, 2024

Addressing feedback from the first draft.

  • Edits and Formalism: align with references/conventions from existing RFCs, references to existing RFCs, clearer definitions
  • Alignment of terms with the direction of industry language. "Privacy policy" versus "Privacy notice"
  • Tool to check and create a privacy.txt policy for any site
  • Acknowledgement notes to feedback and reviews

xcolwell and others added 6 commits May 6, 2024 01:41
- clarify decision to not add api actions from previous author meeting
- split entity per suggestion #2
- add motivation notes for entity
# Conflicts:
#	draft-colwell-privacy-txt.md
@xcolwell
Copy link
Collaborator Author

xcolwell commented May 9, 2024

@lvanderpeet I think the spec should define the encoding of a boolean attribute. Is it 0/1 or false/true?

Alternatively, something to consider, I've been thinking it might be more human readable to work with nominal values instead of booleans. Consider the following two equivalent Cookie lines:

test,foo.com,300,false,true,false,true
test,foo.com,300,first,optional,no-httponly,secure

I feel like the second one is much more readable. What do you think?

Field#4: first or third
Field#5: optional or required
Field#6: httponly or no-httponly
Field#7: secure or no-secure

@xcolwell
Copy link
Collaborator Author

@grittygrease do you have more changes pending? I'm starting to work on fixing the build now.

@xcolwell
Copy link
Collaborator Author

Regarding the URL section, should we just reference RFC3986?

@grittygrease
Copy link
Owner

grittygrease commented Jun 26, 2024 via email

@xcolwell
Copy link
Collaborator Author

xcolwell commented Jun 26, 2024

Merging the General file format, URL, and Email sections together into General File Format with this language makes sense.

I also think a more concise format definition would be UTF-8 encoded (RFC3629) and use \n as a line separator. Multiple whitespace (RFC5892) are collapsed into a single space, and excess space is ignored.

@xcolwell
Copy link
Collaborator Author

An example of merging General file format, URL, and Email into General file format:

The file format is UTF8 text and lists `Field:Value`, one per line. Whitespace and lines that start with `#` are ignored. All field are NOT case sensitive unless mentioned otherwise.  Each field MUST appear on its own line.  Unless otherwise specified by the field definition, multiple values MUST NOT be chained together for a single field. Unless otherwise indicated in a definition of a particular field, a field MAY appear multiple times.

`\n` is used a line break, and whitespace according to the [ASCII whitespace HTML spec](https://infra.spec.whatwg.org/#ascii-whitespace). Excess whitespace is ignored.

Implementors should be aware that some of the fields may contain URIs using percent-encoding (as per Section 2.1 of [RFC3986]) and mailto URIs (as per [RFC6068])

@xcolwell
Copy link
Collaborator Author

@lvanderpeet For this spec NAME, can it just be URL encoded? Is there any reason we need to use a different encoding? Also why a limit on the length?

@xcolwell xcolwell merged commit 7fb8878 into main Jun 26, 2024
1 of 2 checks passed
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.

None yet

3 participants