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.

Sign up for GitHub

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

Jump to bottom

Fix Ceph URL #530

Merged
merged 1 commit into from
Sep 13, 2023
Merged

Fix Ceph URL #530

merged 1 commit into from
Sep 13, 2023

Conversation

Rabenherz112
Copy link
Contributor 

curl -I https://ceph.com/
HTTP/1.1 500 Internal Server Error

curl -I https://ceph.com/en/
HTTP/1.1 200 OK

@Rabenherz112 Rabenherz112 mentioned this pull request Sep 10, 2023
Open
nodiscc
nodiscc reviewed Sep 12, 2023
View reviewed changes
Copy link
Collaborator

@nodiscc nodiscc left a comment

Choose a reason for hiding this comment

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

Well https://ceph.com works for me right now (redirects to https://ceph.com/en/), can you recheck?

@Rabenherz112
Copy link
Contributor Author

The redirection appears to be improperly configured. Although it functions correctly in web browsers, attempting to access it using tools like curl or others leads to a server error (HTTP 500).

I would still be in favour of changing the URL to ensure a positive result from the URL check.

@nodiscc nodiscc merged commit ab95613 into awesome-foss:master Sep 13, 2023
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nodiscc nodiscc nodiscc approved these changes

Assignees
No one assigned
Labels
fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Rabenherz112 @nodiscc