Skip to content
This repository has been archived by the owner on Oct 11, 2021. It is now read-only.
This repository has been archived by the owner on Oct 11, 2021. It is now read-only.

[Requirement] Links must actually go to an existing url #160

Closed
ferventcoder opened this issue Feb 14, 2017 · 5 comments
Closed

[Requirement] Links must actually go to an existing url #160

ferventcoder opened this issue Feb 14, 2017 · 5 comments

Comments

@ferventcoder
Copy link
Member

No description provided.

@mkevenaar
Copy link
Contributor

I would suggest that the package validator requires that all URLs in the nuspec file (e.g. docsUrl, mailinglistUrl) result in a valid HTTP status (2xx, 3xx). A 500 might be considered as a warning, as there could be something wrong with the remote server. Take a look at this page for more status codes.

gep13 added a commit that referenced this issue Dec 23, 2019
@gep13 gep13 closed this as completed Dec 23, 2019
@mkevenaar
Copy link
Contributor

@gep13 the changes you have merged do not validate URL inside markdown fields. Do you still want that?

@gep13
Copy link
Member

gep13 commented Dec 23, 2019

@mkevenaar yes, I think that would make sense.

@mkevenaar
Copy link
Contributor

@gep13 I have opened #196 for this.

@pauby
Copy link
Member

pauby commented Dec 27, 2019

While I was putting together chocochecks I came across issues where valid URL's were actually being flagged as 404's. They work fine in the browser but not using, in my case, Invoke-WebRequest.

What reminded me to come and post this here was this in Gitter.

Unfoirtunately, I don't have an answer for you as I spend a small amount of time looking at it with no result. I haven't had any further time looking at it. So this is more of an FYI / I've seen this issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants