Skip to content

Fix readKey 未指定時に read/getprop が TypeError になる問題 - #6

Open
masaru87 wants to merge 1 commit into
AmbientDataInc:masterfrom
masaru87:fix-readkey-none-guard
Open

Fix readKey 未指定時に read/getprop が TypeError になる問題#6
masaru87 wants to merge 1 commit into
AmbientDataInc:masterfrom
masaru87:fix-readkey-none-guard

Conversation

@masaru87

Copy link
Copy Markdown

read()ambient.py:55)と getprop()(同 81)が readKey の有無を hasattr(self, 'readKey') で判定していますが、__init__self.readKey = readKey(デフォルト None)を常に設定するため、この条件は常に True です。

その結果、readKey を渡さずに Ambient(channelId, writeKey) を生成した場合(公開チャンネルの読み取りなど、readKey が不要な使い方)、'readKey=' + self.readKey'readKey=' + None となり TypeError: can only concatenate str (not "NoneType") to str で失敗します。

Node 版(ambient-nodejs-lib)は同じ箇所を if (this.readKey != null) で判定しており、そちらに合わせて if self.readKey is not None: に修正しました。readKey を指定した場合の挙動は従来どおりです(URL に readKey=... が付与されます)。

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.

1 participant