Skip to content

[KARAF-6652] fixed broken links#1080

Merged
jbonofre merged 1 commit intoapache:masterfrom
oklimberg:KARAF-6652
Mar 28, 2020
Merged

[KARAF-6652] fixed broken links#1080
jbonofre merged 1 commit intoapache:masterfrom
oklimberg:KARAF-6652

Conversation

@oklimberg
Copy link

additionally, using HTTPS for all links

additionally, using HTTPS for all links
@jbonofre jbonofre self-requested a review March 22, 2020 17:15
# Apache Karaf

[Apache Karaf](http://karaf.apache.org) is a modern and polymorphic applications runtime.
[Apache Karaf](https://karaf.apache.org) is a modern and polymorphic applications runtime.
Copy link
Member

Choose a reason for hiding this comment

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

FYI, http://karaf.apache.org is still valid.

Copy link
Member

@jbonofre jbonofre left a comment

Choose a reason for hiding this comment

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

Do you have issue with http URLs ? There are still valid.

@oklimberg
Copy link
Author

oklimberg commented Mar 22, 2020

No, just thought, changing them, so the 'not secure' warning in most of the browsers are gone and users gain trust.
But I can revert that part of the change

@jbonofre
Copy link
Member

Fair enough. Just wondering if you had issue with http.

@jbonofre
Copy link
Member

No your change is fine. Just wanted to check if there were issues with http.

@oklimberg
Copy link
Author

No problem

@jbonofre
Copy link
Member

retest this please

@jbonofre jbonofre merged commit 6010312 into apache:master Mar 28, 2020
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.

3 participants