Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ForwardingFilter に version と metadata を追加する #45

Merged
merged 2 commits into from
Jan 19, 2024

Conversation

miosakuma
Copy link
Contributor

Sora 2023.2.0 の追従で version と metadta を追加しています。

以下の動作確認を行っています。

version

  • 設定したとき、connect に値の送信が行われること
  • 未設定の時、値の送信が行われないこと
  • 文字列でない時に実行エラーとなること

metadata

  • "metadata": "abc" を設定したとき、connect に値の送信が行われること
  • "metadata": {"spam":"egg"} を設定したとき、connect に値の送信が行われること
  • 未設定の時、値の送信が行われないこと
  • JSON 文字列でない時に実行エラーとなること

tests/test_recvonly.py のソースを改変することでテストを行っています

def test_sendonly():
    sora = Sora()

    str_forwarding_filter = """
    {
        "version": "client",
        "metadata": {"spam": "egg", "foo": {"bar": "baz"}},
        "rules": [
            [
                {"field": "kind", "operator": "is_in", "values": ["video"]}
            ]
        ]
    }
    """

    conn = sora.create_connection(
        signaling_urls= ["wss://sora.fine-clue.com/signaling"],
        role="recvonly",
        channel_id="sora",
        forwarding_filter = json.loads(str_forwarding_filter)
    )

filter.version = std::string(object["version"].as_string());
}
if (!object["metadata"].is_null()) {
filter.metadata = object["metadata"];
Copy link
Contributor

@torikizi torikizi Jan 17, 2024

Choose a reason for hiding this comment

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

metadata は他の項目のように ConvertJsonValue をつけなくて良いですか?
signaling_notify_metadata だとこんな感じで設定していたので念のため確認です

  config.signaling_notify_metadata =
      ConvertJsonValue(signaling_notify_metadata,
                       "Invalid JSON value in signaling_notify_metadata");

Copy link
Contributor Author

@miosakuma miosakuma Jan 17, 2024

Choose a reason for hiding this comment

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

確認ありがとうございます。少し前で全体に ConvertJsonValue をかけて変換済みなので問題ない、という理解です。

https://github.com/shiguredo/sora-python-sdk/pull/45/files#diff-0d0290172945fad8d0dd9acd4abac6c388328a3e562f311fcfa83efb79896beaR263

Copy link
Contributor

Choose a reason for hiding this comment

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

おっと見落としておりました。失礼しました。
それであれば大丈夫です。🙏

@miosakuma miosakuma merged commit cc3d0b9 into develop Jan 19, 2024
18 checks passed
@miosakuma miosakuma deleted the feature/add-forwardingfilter-version-metadata branch January 19, 2024 05:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants