Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Remove Wire #1489

Merged
merged 5 commits into from
Nov 19, 2019
Merged

Remove Wire #1489

merged 5 commits into from
Nov 19, 2019

Conversation

xDazld
Copy link
Contributor

@xDazld xDazld commented Nov 14, 2019

Description

Resolves: #1488

Check List

I'm a first time contributor here and likely did something wrong. Please suggest corrections as needed.

@xDazld xDazld requested a review from a team November 14, 2019 03:09
@netlify
Copy link

netlify bot commented Nov 14, 2019

Deploy preview for privacytools-io ready!

Built with commit 2b09ff8

https://deploy-preview-1489--privacytools-io.netlify.com

@nitrohorse
Copy link
Contributor

In case we do end up delisting Wire, we’ll also want to remove the image and Worth Mentioning section under https://deploy-preview-1489--privacytools-io.netlify.com/software/real-time-communication/#teamchat.

@danarel
Copy link
Contributor

danarel commented Nov 14, 2019

I endorse this pull request.

@blacklight447
Copy link
Collaborator

blacklight447 commented Nov 14, 2019

Shouldn't we first figuire the rest out? Removing has quite a dramatic impact, and as long as we dont have a clear picture, people will be wondering what happend .
We did the same with the startpage situation, investigate, document, and remove.

@danarel
Copy link
Contributor

danarel commented Nov 14, 2019

I think this article answers almost all of it. Wire isn’t interested in individual privacy. They even removed the word privacy from their slogan.

In the article, their CEO says their plan all along was to expand enterprise sales and offer security to them and will eventually launch a freemium feature for individuals. They even took down references to the free accounts on the website.

https://techcrunch.com/2019/11/13/messaging-app-wire-confirms-8-2m-raise-responds-to-privacy-concerns-after-moving-holding-company-to-the-us/

@blacklight447
Copy link
Collaborator

I see, that truly is worrying, though i still believe we should put all information on this together to document it, and so we can push out a warning about this today, will you be able to handle this dan?

@dngray
Copy link
Collaborator

dngray commented Nov 14, 2019

I will be working on this issue https://github.com/privacytoolsIO/privacytools.io/issues/1377 in the coming days so I'll take this into account.

The outcome of this should also take into consideration https://github.com/privacytoolsIO/privacytools.io/issues/1437

@xDazld
Copy link
Contributor Author

xDazld commented Nov 14, 2019

The pull doesn't need to be accepted right away. It's just here and ready for if the decision was made to go ahead with removal.
When I get a chance I'll take out the things @nitrohorse mentioned.

privacytoolsIO
privacytoolsIO previously approved these changes Nov 14, 2019
@privacytoolsIO
Copy link
Contributor

I'm in favor of removing it for now. It's fishy and we need to react fast. Doesn't mean that we have to send out warnings or cause any kind of panic at this stage though.

dngray
dngray previously approved these changes Nov 14, 2019
Copy link
Collaborator

@dngray dngray left a comment

Choose a reason for hiding this comment

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

Approving. This is one of the reasons why federated services are a good idea. You pick a server with a privacy policy compatible with your interests.

@blacklight447
Copy link
Collaborator

blacklight447 commented Nov 14, 2019

@BurungHantu1605 removing is more likely to cause panic rather then a small warning sign.

Now i think of it, maybe we should create an "under investigation" status for cases like this. So its not like people have to panic and assume its bad, but have to be cautiones and keep in mind that it may be removed.

@dngray
Copy link
Collaborator

dngray commented Nov 14, 2019

@BurungHantu1605 removing is more likely to cause panic rather then a small warning sign.

I recommend waiting on this as well, approved so just in case we can remove it in a hurry.

This situation has given me the idea to add this point as a disadvantage for centralized systems:

  • Ownership, privacy policy and running of the service can change when a single entity controls it.

@xDazld
Copy link
Contributor Author

xDazld commented Nov 14, 2019

Are we able to specify what exactly are we waiting for? I think it would be best if we can agree on what criteria we still need before the removal is put in place.

Do we want to wait for a specific time period to see what happens? Do we want to see what Wire's next statement will be? Do we want to have someone investigate further into Wire's motives behind all of this?

Copy link
Contributor

@Mikaela Mikaela left a comment

Choose a reason for hiding this comment

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

Change requests: see nitrohorse's comment, I require removing the image and worth mentioning.

As they are doing things better than other examples in the avoid section, I disagree with including Wire there, but if @privacytoolsIO/editorial decides otherwise, I am fine with that.

My opinion: I wish I could disagree with delisting Wire, but I feel disappointed with this new policy, I was advocating for them and moved my family there and have ever overlooked the fact that I have been unable to use Wire's Android app for over a month (wireapp/wire-android#2321).

_includes/sections/instant-messenger.html Outdated Show resolved Hide resolved
_includes/sections/instant-messenger.html Show resolved Hide resolved
@Mikaela
Copy link
Contributor

Mikaela commented Nov 14, 2019

Sorry, I forgot that the Real Time Communication page has three sections from different files. To clarify, what I and @nitrohorse wish you to do is:

  • git rm assets/img/tools/wire.png
  • Remove the worth mentioning section from _includes/sections/teamchat.html (because it would be left empty and bad looking)

Sorry for putting words into your mouth, nitrohorse

@xDazld xDazld dismissed stale reviews from dngray and privacytoolsIO via 3b46e6c November 14, 2019 21:57
The only thing is Wire, which is being removed
@xDazld
Copy link
Contributor Author

xDazld commented Nov 14, 2019

Removed the unused image and worth mentioning as @nitrohorse suggested. Also got rid of a mention of a Wire security audit I missed.
As for adding Wire to the do not use list, the reasoning behind that was to provide a link back to the issue that caused its removal. It was there to solve the problem of a user wondering why Wire vanished. Also, while Wire is open source, it has a policy problem. People do care about that policy problem, that's why this pull request is here. Open to others thoughts on what should be done on that though.

privacytoolsIO
privacytoolsIO previously approved these changes Nov 14, 2019
dngray
dngray previously approved these changes Nov 15, 2019
@Mikaela Mikaela self-requested a review November 15, 2019 16:06
@xDazld xDazld dismissed stale reviews from dngray and privacytoolsIO via 2b09ff8 November 15, 2019 22:59
@dngray dngray dismissed Mikaela’s stale review November 19, 2019 07:07

you're dismissed!

@jonaharagon jonaharagon merged commit 84fbe91 into privacytools:master Nov 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wire ownership changes
8 participants