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

[fetch1075] FetchのTweetがおかしい #1580

Closed
sktometometo opened this issue Sep 1, 2022 · 11 comments
Closed

[fetch1075] FetchのTweetがおかしい #1580

sktometometo opened this issue Sep 1, 2022 · 11 comments
Assignees

Comments

@sktometometo
Copy link
Contributor

Fetch1075のTweetがBase64 encodingされた何かっぽいんですが、これは何かのデバック中ですか?

キャプチャ

@sktometometo
Copy link
Contributor Author

09:30のキッチンデモのDockのタイミングくらいでtweetされてる。

@tkmtnt7000
Copy link
Member

昨日あたりまでは正常にツイートできていそうだったのでおかしいですね。。。午後に確認してみます。

@sktometometo
Copy link
Contributor Author

8/30 にも起きてる。ここ数週間たまにこんな感じで荒ぶっている気がします。最近何かTweet周りいじられてる?

キャプチャ

@sktometometo
Copy link
Contributor Author

8/29 にも。 8/31がたまたま起きなかった?
キャプチャ

@tkmtnt7000
Copy link
Member

僕が最近やっているsmachから通知を行うためにrostwitterを変更していて,ツイートにBase64をを直接送れるようにしています.本来は,画像に変換されて出力されるようにしています.

@sktometometo
Copy link
Contributor Author

了解です、なるほど。

@tkmtnt7000
Copy link
Member

tkmtnt7000 commented Sep 1, 2022

該当となるPRをリンクさせておきます.knorth55/jsk_3rdparty#31, jsk-ros-pkg/jsk_3rdparty#375

@sktometometo
Copy link
Contributor Author

RSJあるので来週以降とかにやりましょう

@tkmtnt7000
Copy link
Member

はい.ちょっと待ち時間がある作業をしたときに確認したのですが,
[ERROR] [1661819674.539695] [/tweet:rosout]: Incorrect padding
おそらくこのエラーが原因と考えています.
base64から画像に変換する際に正しくdecodeできていないことから発生しているようです.
ログ:全体 https://drive.google.com/file/d/10pi7y2w1XHbQrtCz_76kXcFwHnA9k0-E/view?usp=sharing

@iory
Copy link
Member

iory commented Sep 1, 2022

@tkmtnt7000
base64のパースで空白で区切ってもらうという前提を設けました。
こちらのPRで修正しました。
knorth55/jsk_3rdparty#32
tkmtnt7000#8

@sktometometo
Copy link
Contributor Author

Seems to be fixed.

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

No branches or pull requests

3 participants