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

add rule for Merkur.de #6525

Merged
merged 2 commits into from
May 13, 2017
Merged

add rule for Merkur.de #6525

merged 2 commits into from
May 13, 2017

Conversation

numismatika
Copy link
Contributor

No description provided.

@fuglede
Copy link
Contributor

fuglede commented Aug 23, 2016

As far as I can see, this causes problems with video content; on https://www.merkur.de/multimedia/apple-keynote-wwdc-september-2016-iphone-7-zr-6685329.html there is a video missing towards the bottom of the page, due to http://www.snacktv.de/snacktag/snacktag.js being blocked (even though it is actually securable).

@numismatika
Copy link
Contributor Author

SnackTV ruleset update will follow tomorrow.
Meanwhile i asked their customer support about the issue, hopefully it can be properly fixed on their site.

@jeremyn
Copy link
Contributor

jeremyn commented Dec 3, 2016

The embedded YouTube video under "Update vom 8. September 2016:" in https://www.merkur.de/multimedia/apple-keynote-wwdc-september-2016-iphone-7-zr-6685329.html, the URL @fuglede gave, looks okay now. I don't see any other video.

@numismatika Did you hear back from their customer support?

@numismatika
Copy link
Contributor Author

@jeremyn no i didn't. Websites rarely reply to such e-mails i guess.
If they do like the public tv and radio stations "it is work in progress, please wait".
We should check the Outbrain and SnackTV rules though to make sure they do not break

@jeremyn
Copy link
Contributor

jeremyn commented Dec 3, 2016

I'm not sure what you mean. Is there some problem you see that I don't know about?

@numismatika
Copy link
Contributor Author

Not so far

Did you hear back from their customer support?
No and it is brazen to expect them to

@jeremyn
Copy link
Contributor

jeremyn commented Dec 3, 2016

Do you agree that the video problem @fuglede reported is fixed?

@numismatika
Copy link
Contributor Author

looks fine, mixed content warnings do not block the iphone video on the bottom

@jeremyn jeremyn self-assigned this Dec 3, 2016
@jeremyn
Copy link
Contributor

jeremyn commented Dec 18, 2016

Sublist3r reports several more domains, can you add them somewhere please?

www.merkur.de
autoanzeigen.merkur.de
www.autoanzeigen.merkur.de
branchenbuch.merkur.de
epaper.merkur.de
epaperapp.merkur.de
epaperkiosk.merkur.de
immobilien.merkur.de
jobs.merkur.de
markt.merkur.de
www.markt.merkur.de
media.merkur.de
on.merkur.de
playground.merkur.de
promo.merkur.de
www.promo.merkur.de
trauer.merkur.de
www.trauer.merkur.de
veranstaltungen.merkur.de
webviewer.merkur.de

@numismatika numismatika force-pushed the MKD branch 3 times, most recently from bca320e to 940cfdc Compare December 23, 2016 18:44
@jeremyn
Copy link
Contributor

jeremyn commented Mar 8, 2017

Please:

Note as Different HTTP/HTTPS content:

Note as Invalid certificate or Cert mismatch:

Note as Mixed content:

Note as Refused:

Note as Secure connection failed:

Note as Time out:

Add target:

Also:

  • Add the full domain to the items in the comment, for example branchenbuch.merkur.de instead of just branchenbuch
  • Move the https://www.merkur.de target directly below the https://merkur.de target, following our preferred sort order

@numismatika numismatika force-pushed the MKD branch 2 times, most recently from 950ba37 to e3482d1 Compare March 12, 2017 18:58
@numismatika
Copy link
Contributor Author

numismatika commented Mar 13, 2017

@jeremyn on.merkur.de is already covered in the bitly ruleset. No need to duplicate that.

@jeremyn
Copy link
Contributor

jeremyn commented Mar 18, 2017

I've updated the checklist through e3482d1. Can you please double-check your results for https://media.merkur.de and https://jobs.merkur.de ?

- notes about subdomains that do not support tls
@numismatika
Copy link
Contributor Author

media.merkur.de has a cert mismatch , served cert is only valid for tz.addev.de
jobs not even offers a cert.
Nothing changed.

@jeremyn
Copy link
Contributor

jeremyn commented Mar 19, 2017

I can confirm the certificate problem with https://media.merkur.de and I've checked that in the list.

For https://jobs.merkur.de I get "Unable to connect"/"This site can’t be reached", which we label as Refused. This is different from Secure connection failed, which you have labeled it with.

@jeremyn jeremyn closed this Apr 24, 2017
@jeremyn jeremyn reopened this Apr 24, 2017
@jeremyn
Copy link
Contributor

jeremyn commented Apr 25, 2017

@numismatika The only thing stopping me from merging this PR is that you should move https://jobs.merkur.de/ from Secure connection failed to a new Refused category in the top comment.

@jeremyn jeremyn closed this May 13, 2017
@jeremyn jeremyn reopened this May 13, 2017
@jeremyn jeremyn merged commit 7652027 into EFForg:master May 13, 2017
@jeremyn
Copy link
Contributor

jeremyn commented May 13, 2017

I merged this and created pull request #9914 to fix the top comment for https://jobs.merkur.de .

@jeremyn jeremyn removed their assignment May 13, 2017
@numismatika numismatika deleted the MKD branch July 1, 2017 12:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants