Skip to content

Conversation

@ocrybit
Copy link
Contributor

@ocrybit ocrybit commented Feb 13, 2020

概要

  • alis-editorcode-block プラグインを追加することに伴って <pre><code class="language-xxx"> のhtmlタグを許可する必要がある。

関連URL

影響範囲(ユーザ)

  • 全ユーザー

影響範囲(システム)

  • サーバレス
  • フロントエンド
  • エディター

技術的変更点概要

  • html_allowed_tags_v2precode を追加
  • TextSanitizer.allow_code_v2language-xxx フォーマットの class を許可

ユニットテスト

  • test_sanitize_article_body_with_code_unauthorized_class でイリーガルな class をテスト
  • test_sanitize_article_body_v2<pre><code> エレメントを追記してテスト

注意点・その他

  • ローカルにテスト環境の構築ができてないので、実際にテストができていない。

@keillera
Copy link
Member

@ocrybit
こちら確認させて頂きましたが、指摘等ありませんでした。テストケース含めご対応ありがとうございます!!
CI については discord にてご指摘いただいている通り環境設定が必要なので、こちらで確認させて頂きます。

@keillera
Copy link
Member

LGTM です!

一旦サーバサイドのみですがマージさせていただきます!
ご対応頂きありがとうございます 🙏

(補足)

  • CircleCI が失敗しているが、別アカウントにて問題ないことを確認済み(環境変数の問題)
  • フロントエンド側は不具合が見つかっているため別途対応を検討

@keillera keillera merged commit 4757c2a into AlisProject:master Feb 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants