Skip to content
This repository has been archived by the owner on Nov 6, 2022. It is now read-only.

methods: add link and unlink #266

Closed
wants to merge 1 commit into from
Closed

Conversation

jasnell
Copy link
Member

@jasnell jasnell commented Sep 27, 2015

LINK and UNLINK are registered methods and are in real world use.

@@ -123,7 +123,7 @@ do { \
FOR##_mark = NULL; \
} \
} while (0)

This comment was marked as off-topic.

This comment was marked as off-topic.

@indutny
Copy link
Member

indutny commented Sep 27, 2015

@jasnell are these WebDAV methods? Where are they specified?

@jasnell
Copy link
Member Author

jasnell commented Sep 27, 2015

Originally, in RFC2068. They were pulled from 2616 because of lack of use.
I introduced a new I-D to update their definition and RFC 7237 added them
to the iana registry. I had let my ID expire but recently found some real
world implementation that led me to update to ID.
On Sep 27, 2015 12:57 PM, "Fedor Indutny" notifications@github.com wrote:

@jasnell https://github.com/jasnell are these WebDAV methods? Where are
they specified.


Reply to this email directly or view it on GitHub
#266 (comment).

LINK and UNLINK are registered methods and are in real world use.
@jasnell
Copy link
Member Author

jasnell commented Sep 27, 2015

@indutny ... ok, unnecessary style changes removed.

@indutny
Copy link
Member

indutny commented Oct 7, 2015

Test?

@jasnell
Copy link
Member Author

jasnell commented Oct 7, 2015

I'll get one added!
On Oct 7, 2015 9:11 AM, "Fedor Indutny" notifications@github.com wrote:

Test?


Reply to this email directly or view it on GitHub
#266 (comment).

@dolmen
Copy link
Contributor

dolmen commented Oct 12, 2015

I just submitted #267 that has tests.

@jasnell
Copy link
Member Author

jasnell commented Oct 12, 2015

@dolmen ... thank you!

@jasnell
Copy link
Member Author

jasnell commented Oct 26, 2015

Closed in favor of #267

@jasnell jasnell closed this Oct 26, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants