treehouse
/
mastodon
21
8
Fork 5
Code Issues 32 Pull Requests Projects 1 Releases Wiki Activity

Decode IDNA in PreviewCard (#2781)

lolsob-rspec
Yamagishi Kazutoshi 2017-05-04 22:52:08 +09:00 committed by Eugen Rochko
parent 6bbf4b89bb
commit 93ec05f976
5 changed files with 23 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
app/javascript/mastodon/features/status/components/card.js
View File

@ -1,11 +1,15 @@
import React from 'react'; import React from 'react';
import ImmutablePropTypes from 'react-immutable-proptypes'; import ImmutablePropTypes from 'react-immutable-proptypes';
import punycode from 'punycode'
const hostStyle = { const IDNA_PREFIX = 'xn--';
display: 'block',
marginTop: '5px', const decodeIDNA = domain => {
fontSize: '13px' return domain
}; .split('.')
.map(part => part.indexOf(IDNA_PREFIX) === 0 ? punycode.decode(part.slice(IDNA_PREFIX.length)) : part)
.join('.');
}
const getHostname = url => { const getHostname = url => {
const parser = document.createElement('a'); const parser = document.createElement('a');
@ -30,7 +34,7 @@ class Card extends React.PureComponent {
} }
if (provider.length < 1) { if (provider.length < 1) {
provider = getHostname(card.get('url')) provider = decodeIDNA(getHostname(card.get('url')));
} }
return ( return (
@ -40,7 +44,7 @@ class Card extends React.PureComponent {
<div className='status-card__content'> <div className='status-card__content'>
<strong className='status-card__title' title={card.get('title')}>{card.get('title')}</strong> <strong className='status-card__title' title={card.get('title')}>{card.get('title')}</strong>
<p className='status-card__description'>{(card.get('description') || '').substring(0, 50)}</p> <p className='status-card__description'>{(card.get('description') || '').substring(0, 50)}</p>
<span className='status-card__host' style={hostStyle}>{provider}</span> <span className='status-card__host'>{provider}</span>
</div> </div>
</a> </a>
); );

6
app/javascript/styles/components.scss
View File

@ -1818,6 +1818,12 @@ button.icon-button.active i.fa-retweet {
color: $color3; color: $color3;
} }
.status-card__host {
display: block;
margin-top: 5px;
font-size: 13px;
}
.status-card__image { .status-card__image {
flex: 0 0 100px; flex: 0 0 100px;
background: lighten($color1, 8%); background: lighten($color1, 8%);

1
app/services/fetch_link_card_service.rb
View File

@ -11,6 +11,7 @@ class FetchLinkCardService < BaseService
return if url.nil? return if url.nil?
url = Addressable::URI.parse(url).normalize.to_s
card = PreviewCard.where(status: status).first_or_initialize(status: status, url: url) card = PreviewCard.where(status: status).first_or_initialize(status: status, url: url)
attempt_opengraph(card, url) unless attempt_oembed(card, url) attempt_opengraph(card, url) unless attempt_oembed(card, url)
end end

1
package.json
View File

@ -63,6 +63,7 @@
"postcss-smart-import": "^0.6.12", "postcss-smart-import": "^0.6.12",
"precss": "^1.4.0", "precss": "^1.4.0",
"prop-types": "^15.5.8", "prop-types": "^15.5.8",
"punycode": "^2.1.0",
"rails-erb-loader": "^5.0.0", "rails-erb-loader": "^5.0.0",
"react": "^15.5.4", "react": "^15.5.4",
"react-addons-perf": "^15.4.2", "react-addons-perf": "^15.4.2",

4
yarn.lock
View File

@ -5343,6 +5343,10 @@ punycode@^1.2.4, punycode@^1.4.1:
version "1.4.1" version "1.4.1"
resolved "https://registry.yarnpkg.com/punycode/-/punycode-1.4.1.tgz#c0d5a63b2718800ad8e1eb0fa5269c84dd41845e" resolved "https://registry.yarnpkg.com/punycode/-/punycode-1.4.1.tgz#c0d5a63b2718800ad8e1eb0fa5269c84dd41845e"
punycode@^2.1.0:
version "2.1.0"
resolved "https://registry.yarnpkg.com/punycode/-/punycode-2.1.0.tgz#5f863edc89b96db09074bad7947bf09056ca4e7d"
q@^1.1.2: q@^1.1.2:
version "1.4.1" version "1.4.1"
resolved "https://registry.yarnpkg.com/q/-/q-1.4.1.tgz#55705bcd93c5f3673530c2c2cbc0c2b3addc286e" resolved "https://registry.yarnpkg.com/q/-/q-1.4.1.tgz#55705bcd93c5f3673530c2c2cbc0c2b3addc286e"