Skip to content
This repository was archived by the owner on Nov 6, 2023. It is now read-only.

Add Net-metrix.ch.xml and update Wemfbox.ch.xml - #14330

Merged
zoracon merged 7 commits into
EFForg:masterfrom
J0WI:wemfbox
Nov 14, 2018
Merged

Add Net-metrix.ch.xml and update Wemfbox.ch.xml#14330
zoracon merged 7 commits into
EFForg:masterfrom
J0WI:wemfbox

Conversation

@J0WI

@J0WI J0WI commented Jan 18, 2018

Copy link
Copy Markdown
Contributor

Wemfbox.ch.xml is only tracking anyway.
See also: #14293

@jeremyn

jeremyn commented Jan 21, 2018

Copy link
Copy Markdown
Contributor

Where are you getting information about the redirects? For example how would I verify this change is correct?

		<rule from="^http://wetter\.wemfbox\.ch/"
		to="https://we-ssl.wemfbox.ch/" />

to

		<rule from="^http://wirelter\.wemfbox\.ch/"
		to="https://we-ssl.wemfbox.ch/" />

Neither wetter.wemfbox.ch nor wirelter.wemfbox.ch redirect themselves to we-ssl.wemfbox.ch.

@J0WI

J0WI commented Jan 21, 2018

Copy link
Copy Markdown
Contributor Author

I was searching for the source of wirelter.wemfbox.ch (which I found in an archived version of the site) and found we-ssl.wemfbox.ch in the current source code of https://www.wireltern.ch/.

@cschanaj

Copy link
Copy Markdown
Collaborator

related to #14660

@Bisaloo

Bisaloo commented Mar 21, 2018

Copy link
Copy Markdown
Collaborator

Related: #12297

@J0WI

J0WI commented Jun 16, 2018

Copy link
Copy Markdown
Contributor Author

I just rebased this. Would be great to get a review here to fix the mentioned issues.

@zoracon
zoracon self-requested a review November 14, 2018 20:58

@zoracon zoracon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@zoracon
zoracon merged commit 5d1c6ad into EFForg:master Nov 14, 2018
@J0WI
J0WI deleted the wemfbox branch November 14, 2018 23:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants