Prefix env var with COMM_

added @ashoat since this is a new dependency.

otherwise looks fine

COMM_JSONCONFIG variables are never file paths, they are always JSON blobs.

Yeah, that's what Shellcheck wanted me to do. But I think it's unsafe - doesn't that mean $COMM_JSONCONFIG_secrets_geoip_license won't be quoted? I think the open quote is actually closing the quote from line 13, and the close quote is actually opening a new quote. So that leaves $COMM_JSONCONFIG_secrets_geoip_license unquoted

Going to assume that we don't want to spend anymore time on this.

switched [[ -n to [[ -r, as I realized later that COMM_JSONCONFIG_secrets_geoip_license was a file path to a json config.

Capitalize docker, revisit error message

Address feedback: move secrets dir creation to it's own block

Rebase on master

Address feedback

Address feedback

Looks fine for a transition commit, still missing the other X3DH elements.

Address feedback

Address feedback

looks good to me from a bash perspective

Looks like i got distracted friday

Remove primaryDevice, reference protobuf file

Address more feedback

Add device type

rust looks okay to me. Just don't like that the callee has to have awareness of the promise ID.

address feedback

Remove index.js stub

LGTM, ill defer to react/redux people for correctness

I don't feel strongly about the rust code, which is ephemeral anyway.

I'm not able to comment on gradle code, but I couldn't find a better way to do this using just cmake + corrosion + cxxbridge

After running yarn arcpatch D7197, I noticed that the commit before this one is D7155. Does this work depend on D7155? If so, can you specify the stack on Phabricator?

Nothing looks super surprising here, thanks!

A little out my expertise for best practcies, but looks good

Rebase on master