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

HTTP -> HTTPS except POM headers and Maven Wrapper files (#120) #175

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

LIttleAncientForestKami

Overview

  • Java files have Javadocs changed: URLs now use https
  • grep http **/* -r | grep -v https was used for testing / looking for places to change
  • all links I changed worked in their changed version
  • the issue says (in bold even) all files, so do let me know if it really is what was meant @sormuras and @marcphilipp - I didn't fully follow, since:
  • Maven wrapper files are IIRC autogenerated, so I did not make changes there
  • Maven POM documentation and example use http for XML namespaces in the headers, and both http and https link fail their resolution. Example docs: https://maven.apache.org/pom.html - so XML headers I made as Maven docs have them, in case some legacy tooling needs that.

Let me know what else's needed.

I hereby agree to the terms of the JUnit Contributor License Agreement.

)

The C Programming Language — A language which combines the flexibility of assembly language with the power of assembly language.
Copy link
Member

@sormuras sormuras left a comment

Choose a reason for hiding this comment

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

Thanks for submitting this PR Tomasz - please address the request changes.

junit5-jupiter-starter-maven/pom-SNAPSHOT.xml Show resolved Hide resolved
junit5-jupiter-starter-maven/pom.xml Show resolved Hide resolved
@sbrannen
Copy link
Member

Thanks for the PR.

It would be better to introduce nohttp into the build(s) instead of manually grepping.

Perhaps you'd like to introduce nohttp with this PR.

@sbrannen
Copy link
Member

@sormuras
Copy link
Member

I guess, I found another candidate for adding a s to right URI:

xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">

Is there a standalone nohttp variant, not needing a build tool integreation?
Asking for "ant", "bazel", and "modular-world" samples.

@sbrannen
Copy link
Member

I guess, I found another candidate for adding a s to right URI:

nohttp should inform us about everything this PR misses, and it will do that for every build.

Is there a standalone nohttp variant, not needing a build tool integreation?

Yes. Check out the project page. There's a CLI, Checkstyle support, and Gradle support.

https://github.com/spring-io/nohttp#modules

@LIttleAncientForestKami
Copy link
Author

@sbrannen
Copy link
Member

I just became a father!!

Congratulations! 👶

Nohttp sounds like just what is needed and remarks about XML headers are on point.

👍

Once I'm done should I squash the commits?

Yes, that would be fine. Feel free to rename the issue, rework the description, squash commits, rebase on main, and force push the changes to your PR branch.

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.

None yet

3 participants