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

Fix links #1708

Merged
merged 1 commit into from
Feb 16, 2024
Merged

Fix links #1708

merged 1 commit into from
Feb 16, 2024

Conversation

tsalo
Copy link
Member

@tsalo tsalo commented Feb 15, 2024

Closes none. Fixes the links that are causing the link-checker CI step to fail.

@tsalo tsalo added exclude-from-changelog This item will not feature in the automatically generated changelog documentation labels Feb 15, 2024
@tsalo
Copy link
Member Author

tsalo commented Feb 15, 2024

The INCF website error seems to stem from calling the URL too many times, rather than a broken URL.

@effigies
Copy link
Collaborator

The INCF issue is that they seem to be signed with a certificate that isn't in OS certificate stores yet. Try running curl https://www.incf.org.

@effigies effigies merged commit fd6612e into bids-standard:master Feb 16, 2024
24 of 25 checks passed
@effigies
Copy link
Collaborator

Opened a thread at https://neurostars.org/t/incf-website-not-sending-a-complete-certificate-chain/28163. I don't know if either of you have a better notion where to ping INCF folks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation exclude-from-changelog This item will not feature in the automatically generated changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants