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

Specify some normalization rules for URIs #147

Closed
wants to merge 1 commit into from

Conversation

adrianhopebailie
Copy link
Collaborator

No description provided.

@@ -1211,7 +1211,9 @@ <h1 id="rfc.section.9.2.1"><a href="#rfc.section.9.2.1">9.2.1.</a> <a href="#par
<h1 id="rfc.section.9.2.2"><a href="#rfc.section.9.2.2">9.2.2.</a> <a href="#parameter-cost-cost" id="parameter-cost-cost">Parameter: Cost (cost)</a></h1>
<p id="rfc.section.9.2.2.p.1">The cost parameter is the cost of the condition that is represented by the URI.</p>
<h1 id="rfc.section.9.2.3"><a href="#rfc.section.9.2.3">9.2.3.</a> <a href="#parameter-subtypes-subtypes" id="parameter-subtypes-subtypes">Parameter: Subtypes (subtypes)</a></h1>
<p id="rfc.section.9.2.3.p.1">The subtypes parameter indicates the types of conditions that are subtypes of the condition represented by the URI. The value MUST be a comma seperated list of names from the <a href="#crypto-conditions-type-registry">Crypto-Condition Type Registry</a> <cite title="NONE">[crypto-conditions-type-registry]</cite> and SHOULD not include the type of the root crypto-condition. i.e. The value of the fpt paramtere should not appear in the list of types provided as the value of the subtypes parameter.</p>
<p id="rfc.section.9.2.3.p.1">The subtypes parameter indicates the types of conditions that are subtypes of the condition represented by the URI. The value MUST be a comma seperated list of names from the <a href="#crypto-conditions-type-registry">Crypto-Condition Type Registry</a> <cite title="NONE">[crypto-conditions-type-registry]</cite>.</p>
Copy link
Member

Choose a reason for hiding this comment

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

seperated should be separated

@@ -457,7 +457,7 @@

<meta name="dct.creator" content="Thomas, S., Reginelli, R., and A. Hope-Bailie" />
<meta name="dct.identifier" content="urn:ietf:id:draft-thomas-crypto-conditions-02" />
Copy link
Member

Choose a reason for hiding this comment

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

This is going to be draft-03 now, right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Correct! I actually think we should move this whole spec to it's own repo to allow it to be tagged before merging any more changes.

See #148

adrianhopebailie added a commit to rfcs/crypto-conditions that referenced this pull request Mar 14, 2017
adrianhopebailie added a commit to rfcs/crypto-conditions that referenced this pull request Mar 14, 2017
@adrianhopebailie
Copy link
Collaborator Author

Migrated to rfcs/crypto-conditions

@adrianhopebailie adrianhopebailie deleted the ahb-uri-normalization branch March 14, 2017 14:46
adrianhopebailie added a commit to rfcs/crypto-conditions that referenced this pull request Jul 13, 2017
adrianhopebailie added a commit to rfcs/crypto-conditions that referenced this pull request Jul 13, 2017
Migrate PRs from:
interledger/rfcs#147
interledger/rfcs#157

* Update draft number
* Make commas reserved characters in URI
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.

2 participants