Tom_Eagle
8. März 2021 um 20:10
1
Hallo zusammen,
in unserem Forum gibt es ein Problem: Wenn eine URL in eine „Onebox
Richie
8. März 2021 um 20:19
2
Diese beiden Links sind identisch:
https://www.morphmarket.com/c/reptiles/pythons/ball-pythons/genetic-calculator/?s1=Pastel,Het%2BClown&s2=Pastel,Het%2BClown
Könntest du sie in Backticks setzen, damit wir sie so sehen können, wie du es beabsichtigt hast?
Tom_Eagle
8. März 2021 um 20:58
3
Entschuldigung, mein Fehler
So sollte es aussehen:
https://www.morphmarket.com/c/reptiles/pythons/ball-pythons/genetic-calculator/?s1=Pastel%2CHet+Clown&s2=Pastel%2CHet+Clown
So sieht es aus, wenn es einzeilig eingebunden wird:
https://www.morphmarket.com/c/reptiles/pythons/ball-pythons/genetic-calculator/?s1=Pastel,Het%2BClown&s2=Pastel,Het%2BClown
Wenn die URL in beiden Fällen funktioniert, warum ist es dann wichtig?
Tom_Eagle
9. März 2021 um 11:25
5
Weil sie zu unterschiedlichen Ergebnissen führen.
Hier ist, was über den Link angezeigt werden sollte:
Nicht dies:
Dies macht einen großen Unterschied, wenn man die Genetik korrekt berechnen möchte.
david
24. Juli 2023 um 11:59
9
Es scheint, dass die URLs im OP nicht mehr oneboxable sind (keine Opengraph-Tags auf der Website). Aber nur zur Information, dieses Problem sollte behoben worden sein durch
committed 10:55AM - 09 Aug 22 UTC
normalized_encode in addressable has a number of issues, including https://githu… b.com/sporkmonger/addressable/issues/472
To temporaily work around those issues for the majority of cases, we try parsing with `::URI`. If that fails (e.g. due to non-ascii characters) then we will fall back to addressable.
Hopefully we can simplify this back to `Addressable::URI.normalized_encode` in the future.
This commit also adds support for unicode domain names and emoji domain names with escape_uri.
This removes an unneeded hack checking for pre-signed urls, which are now handled by the general case due to starting off valid and only being minimally normalized. Previous test case continues to pass.
UrlHelper.s3_presigned_url? which was somewhat wide was removed.
