Skip to content
This repository has been archived by the owner on Feb 12, 2023. It is now read-only.

Verify our compliance with the Tox Client Standard #5243

Open
sudden6 opened this issue Jul 14, 2018 · 1 comment · May be fixed by #5244
Open

Verify our compliance with the Tox Client Standard #5243

sudden6 opened this issue Jul 14, 2018 · 1 comment · May be fixed by #5244
Labels
C-feature-request The issue contains a feature request D-easy We estimate that the issue is easy to fix M-architecture Category for internal architecture problems

Comments

@sudden6
Copy link
Member

sudden6 commented Jul 14, 2018

Since the plan is to remove clients which don't comply to TCS from tox.chat, we should verify we do in fact comply with TCS.

#5241 adds a new document where we keep track of the current state.

@sudden6 sudden6 added D-easy We estimate that the issue is easy to fix M-architecture Category for internal architecture problems labels Jul 14, 2018
@tox-user tox-user linked a pull request Jul 15, 2018 that will close this issue
@jessica181920
Copy link
Contributor

jessica181920 commented Mar 15, 2019

I think qtox is in pretty decent shape. I can't verify all of them, but it seems the majority are okay. Just browsing through here's a few I noticed that qtox fails:

2.2.6 A client may only transmit avatars with an aspect ratio of 1:1. If the user has chosen a non-square avatar, the client must transmit the square portion of the avatar displayed to the user. In the case that the user's client doesn't display avatars, the client must pad the avatar evenly on two sides with transparent rectangles, in order to create a square.

3.5.1 A client must encode emoticons as Unicode emoji.

4.0.2 If message persistence is enabled by default, all message logs stored on the device must be encrypted using the cryptography functions provided by the Tox API, or a comparably reputable, free and open source cryptography library.

3.5.1 is a really easy fix, delete "Basic," "Classic," "emojione," and "Universe" and rename "ASCII+emojione" to "emojione" and "ASCII+Universe" to "twemoji." The "ASCII+" emoticon packs, ironically and oddly enough, do not have any ASCII emotes so they should be ok. but they do have :emote: style emoticons.

@sudden6 sudden6 added the C-feature-request The issue contains a feature request label Aug 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C-feature-request The issue contains a feature request D-easy We estimate that the issue is easy to fix M-architecture Category for internal architecture problems
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants