Page MenuHomePhorge

avivey (Aviv Eyal)
UserAdministrator

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
Jun 10 2021, 05:36 (76 w, 5 d)
Roles
Administrator
Availability
Available

Recent Activity

Fri, Nov 18

avivey added a comment to Q26: Javascript error on main page.

I don't see this error - can you provide reproduction instructions?

Fri, Nov 18, 06:58

Thu, Nov 3

avivey added a comment to Q21: Only the last 4 diffs are visible in the changed files history. .

That's usually a sign that the Daemons are in some trouble. Go to the Daemons app and see their status and task status.

Thu, Nov 3, 15:34

Oct 29 2022

avivey added a comment to T15106: "Upload file" in remarkup text fields should attach by default .

For completeness: The security breaches mentioned in the security guidance task are all about exposing an existing file that the attacker has new view access to; This flow is only relevant to explicitly uploading a new file, so it doesn't have the same security implications.

Oct 29 2022, 07:07 · Remarkup

Oct 28 2022

avivey added a comment to D25054: Replace PHPExcel with PhpSpreadsheet.

The change should also include updating the installation instructions, and a ticket.

Oct 28 2022, 10:55
avivey accepted D25057: updating twitch to latest api (Helix).
Oct 28 2022, 10:53 · Auth
avivey added a comment to T15133: Differential List View should show Repository.

https://secure.phabricator.com/T9782 suggests it should be using the "custom fields" interface and eventually fizzles out.

Oct 28 2022, 10:48 · Differential
avivey updated the diff for D25057: updating twitch to latest api (Helix).

Rebased and run unit tests again

Oct 28 2022, 10:43 · Auth
avivey accepted D25056: updating twitch to latest api (Helix).

(On a broader theme, this code should probably move into rP, right? Is there any use-case for arcanist directly making calls to 3rd party services?)

Oct 28 2022, 10:34 · Auth
avivey added a comment to T15122: Twitch auth provider is not working - old twitch API has been decomissioned.

running celerity map should just fix it. I'll take a look.

Oct 28 2022, 10:32 · Auth
avivey created T15133: Differential List View should show Repository.
Oct 28 2022, 10:31 · Differential
avivey accepted D25037: Hide the "hidden" fields on custom form previews.

I'm good with this, but we have a policy of "no landed diffs w/o tickets", so please create one before landing.

Oct 28 2022, 10:28
avivey accepted D25055: Update sprite sheets.

We should connect this script (scripts/celerity/*.php) into bin/celerity.

Oct 28 2022, 10:26 · Auth
avivey accepted D25052: Add and use new RemarkupMetadata class.
Oct 28 2022, 10:19 · Remarkup
avivey added a member for Trusted Contributors: cristian64.
Oct 28 2022, 07:49
avivey created T15132: Support monospaced links in remarkup.
Oct 28 2022, 07:48 · Remarkup
avivey created Remarkup.
Oct 28 2022, 07:38

Oct 9 2022

avivey closed D25032: Update PassphraseCredential to support Mentions.
Oct 9 2022, 10:30
avivey committed rP4aedc206c727: Update PassphraseCredential to support Mentions (authored by Ekubischta).
Update PassphraseCredential to support Mentions
Oct 9 2022, 10:30
avivey closed D25021: Added cross-platform default fonts.
Oct 9 2022, 10:30
avivey closed T15049: Segoe UI Symbol provides bad user experience as Resolved by committing rPf8ffa393c442: Added cross-platform default fonts.
Oct 9 2022, 10:30
avivey committed rPf8ffa393c442: Added cross-platform default fonts (authored by vedranmiletic).
Added cross-platform default fonts
Oct 9 2022, 10:30
avivey changed the visibility for Security.
Oct 9 2022, 10:20
avivey accepted D25021: Added cross-platform default fonts.
Oct 9 2022, 10:16
avivey created an object: 2022-10-04.
Oct 9 2022, 10:04

Oct 4 2022

avivey added a comment to T15127: Allow for reordering of milestones.

The original discussion in T12144 is kinda convincing to me, although it's mostly convinces me that the term "milestone" doesn't match these objects.

Oct 4 2022, 07:00 · Projects
avivey added a comment to T15126: Remove Chatlog.

can we keep the actual deletion as optional for a very long time/forever, and at the same time not create the database for new installs?
that will allow installs to backup/export their logs even if they miss the deprecation warning.

Oct 4 2022, 06:33 · Chatlog

Sep 28 2022

avivey added a comment to T15122: Twitch auth provider is not working - old twitch API has been decomissioned.

also link to the updated Twitch docs about the auth API?

Sep 28 2022, 18:15 · Auth
avivey added a comment to T15122: Twitch auth provider is not working - old twitch API has been decomissioned.

@k__nard: do you mind submitting the diffs as revisions using arc diff?

Sep 28 2022, 18:12 · Auth

Sep 27 2022

avivey requested changes to D25020: add --verbose argument to ArcanistLiberateWorkflow.

Looks like https://secure.phabricator.com/D21718 explicitly changed to always remove --quiet, so unless there's a new argument for it, this change is redundant.

Sep 27 2022, 15:52
avivey accepted D25021: Added cross-platform default fonts.

I've ran this locally, and it looks the same to me (Chrome on Windows).

Sep 27 2022, 14:45
avivey updated the diff for D25021: Added cross-platform default fonts.

I've re-applied this patch and uploaded with arc diff, just to normalize it.

Sep 27 2022, 14:40
avivey added a comment to D25052: Add and use new RemarkupMetadata class.

Bear with me here, because Javeline (like all js frameworks) is confusing to me.

Sep 27 2022, 14:23 · Remarkup
avivey updated the task description for T15123: Javeline Support stuff.
Sep 27 2022, 09:17
avivey created T15124: arc-browse is broken here.
Sep 27 2022, 09:12 · phorge.it install
avivey added a comment to T15123: Javeline Support stuff.
Sep 27 2022, 09:09
avivey claimed T15123: Javeline Support stuff.
Sep 27 2022, 09:05

Sep 25 2022

avivey added a project to T15122: Twitch auth provider is not working - old twitch API has been decomissioned: Auth.
Sep 25 2022, 04:51 · Auth

Sep 23 2022

avivey added a comment to T15106: "Upload file" in remarkup text fields should attach by default .

@kwisatz I'll try to review the diff this week, and that will probably get to master/stable soon...

Sep 23 2022, 07:12 · Remarkup
avivey created an object: 2022-09-20.
Sep 23 2022, 06:53
avivey triaged T15121: Contributor Agreement as Unbreak Now! priority.

As in E14, this is really important for protecting us and contributors.

Sep 23 2022, 06:45 · Phorge
avivey edited the content of Planning Meetings.
Sep 23 2022, 06:10 · Phorge Upstream

Sep 22 2022

avivey added a comment to T15026: Create a migration guide to move from Phabricator to Phorge.

Wouldn't it just be easier for users if the current Phacility/stable was merged into Phorge/stable, thus avoiding the need to issue git reset --hard?

Sep 22 2022, 18:41 · Phorge
avivey added a comment to T15026: Create a migration guide to move from Phabricator to Phorge.

we've decided to add git reset --hard to the instructions to avoid the merges and simplify the stable issue, avoiding the merge from multiple branches.

Sep 22 2022, 13:56 · Phorge

Sep 20 2022

avivey renamed T15005: Get Better Logo / Brand Assets from Create Logo / Brand Assets to Get Better Logo / Brand Assets.
Sep 20 2022, 20:12 · Phorge
avivey claimed T15077: Rebrand: Tracking task.

I'll go over this to see if there's anything left to do.

Sep 20 2022, 19:39 · Phorge
avivey added a comment to Update From Phabricator.

What if you upgrade a heavily modified (forked) Phabricator to the vanilla Phorge?

Sep 20 2022, 19:27
avivey updated the post content for Blog Post: Going Public.
Sep 20 2022, 19:25
avivey updated the post content for Blog Post: Going Public.
Sep 20 2022, 19:19
avivey created T15119: 502 upstream sent too big header while.
Sep 20 2022, 19:08 · phorge.it install
avivey edited the content of Update From Phabricator.
Sep 20 2022, 07:47
avivey added a comment to Update From Phabricator.

ah, I see the error now.
yes, that's a typo - it was carried over from an earlier version of the document.
in phase 5, only node-modules should be removed, not the package files.
thank you for reporting, and sorry I was being harsh - I forgot I split this section and only looked at the primary blob of commands.

Sep 20 2022, 07:46
avivey added a comment to Update From Phabricator.

@kwisatz - did you actually try to follow the instructions and ran into a problem, or are you just speculating?

Sep 20 2022, 07:27
avivey added a comment to T15073: /status should be always allowed in anonymous mode.

I just tried it here, and it didn't require any auth. maybe something was wrong with our setup at the time?

Sep 20 2022, 06:48 · Phorge General/Unknown

Sep 19 2022

avivey closed T15023: Create Phorge, a fork of Phabricator as Resolved.

We've made it to y-combinator, so I think that counts.

Sep 19 2022, 19:04 · Phorge
avivey closed T15117: 404 on links to we.phorge.it links from the phorge.it homepage as Resolved.

Fixed and deployed.

Sep 19 2022, 19:02
avivey added a comment to T15012: Update Diviner documentation to reference Phorge.

I think we can close this now.

Sep 19 2022, 19:02 · Phorge
avivey closed T15012: Update Diviner documentation to reference Phorge, a subtask of T15006: Re-brand Phorge, as Resolved.
Sep 19 2022, 19:01 · Phorge
avivey closed T15012: Update Diviner documentation to reference Phorge as Resolved.
Sep 19 2022, 19:01 · Phorge
avivey closed D25050: Update Diviner links.
Sep 19 2022, 19:00 · Diviner
avivey committed rHP7ec5b900ae3d: Update Diviner links (authored by avivey).
Update Diviner links
Sep 19 2022, 19:00
avivey added a comment to Update From Phabricator.

Why do we run npm install when we have previously deleted the package.json file from that directory?

Sep 19 2022, 18:28

Sep 16 2022

avivey added a comment to T15117: 404 on links to we.phorge.it links from the phorge.it homepage .

thanks - not technically a dupe, because there's no ticket, but D25050 would fix those.

Sep 16 2022, 17:28
avivey added Q16: How to use selfhosted Divine instead of Phabricator? (Answer 9).
Sep 16 2022, 06:08

Sep 9 2022

dtf awarded Blog Post: Going Public a Party Time token.
Sep 9 2022, 19:07
avivey added a comment to T15092: Automatic deploy onto we.phorge.it.

(should also include the rHP -> /home/ stuff)

Sep 9 2022, 13:09 · phorge.it install, Upstream General/Unknown
avivey added a reviewer for D25050: Update Diviner links: Blessed Committers.
Sep 9 2022, 13:08 · Diviner
avivey requested review of D25050: Update Diviner links.
Sep 9 2022, 13:08 · Diviner
avivey added Q14: How ready is Phorge to replace stable versions of Phabricator? (Answer 8).
Sep 9 2022, 13:04
avivey added a comment to T15012: Update Diviner documentation to reference Phorge.

Done - deleted phabricator books.

Sep 9 2022, 13:01 · Phorge
avivey added Q15: Can you allow git fetching specific commits? (Answer 7).
Sep 9 2022, 12:55
avivey added a comment to T15026: Create a migration guide to move from Phabricator to Phorge.

I've updated the guide with @jeremy.norris's Aphlict instructions, and also a section about configurations we might change later.

Sep 9 2022, 08:43 · Phorge
avivey edited the content of Update From Phabricator.
Sep 9 2022, 08:40
avivey edited the content of 2022 Week 37.
Sep 9 2022, 08:37
avivey updated the task description for T15026: Create a migration guide to move from Phabricator to Phorge.
Sep 9 2022, 07:37 · Phorge

Sep 8 2022

avivey edited the content of Update From Phabricator.
Sep 8 2022, 18:16
avivey edited the content of 2022 Week 37.
Sep 8 2022, 18:12
avivey edited the content of Update From Phabricator.
Sep 8 2022, 18:08
avivey added a comment to T15026: Create a migration guide to move from Phabricator to Phorge.

the Aphlict change sounds simple enough to just add to the migration guide - it's going to happen at the same time for basically everyone anyway.

Sep 8 2022, 18:07 · Phorge
avivey created an object: 2022-09-06.
Sep 8 2022, 18:02
avivey updated the task description for T15116: can't email replies .
Sep 8 2022, 17:42 · phorge.it install
avivey added a comment to T15026: Create a migration guide to move from Phabricator to Phorge.

re: stable, I'm not sure how the commits actually relate between the now 4 branches.
I considered adding git reset --hard for stable, but I was afraid users will lose local changes.
So you have a suggestion on how to fix the guide for stable?

Sep 8 2022, 17:39 · Phorge
avivey updated the post content for Blog Post: Going Public.
Sep 8 2022, 17:29
avivey updated the post content for Blog Post: Going Public.
Sep 8 2022, 17:27
avivey added a comment to T15046: Add GitHub mirrors for repos.

need to create a username/password passphrase credential with creds for a GitHub account that has write access to the GitHub repos, and set it in the diffusion uris.

I was hoping that wouldn't be the case. I'll create a credential against my account, since it's currently impossible to create GitHub account keys against an orginization.

Sep 8 2022, 17:20 · Nuance, Phorge Upstream, Diffusion
avivey claimed T15116: can't email replies .
Sep 8 2022, 11:35 · phorge.it install
avivey created T15116: can't email replies .
Sep 8 2022, 11:35 · phorge.it install

Sep 7 2022

avivey added a comment to T15046: Add GitHub mirrors for repos.
Sep 7 2022, 18:17 · Nuance, Phorge Upstream, Diffusion
avivey updated the task description for T15046: Add GitHub mirrors for repos.
Sep 7 2022, 18:16 · Nuance, Phorge Upstream, Diffusion

Sep 6 2022

avivey added a comment to T15046: Add GitHub mirrors for repos.

We've created https://github.com/phorgeit and started configuring it to mirror the repos.

Sep 6 2022, 19:27 · Nuance, Phorge Upstream, Diffusion

Sep 5 2022

avivey added a comment to Q15: Can you allow git fetching specific commits?.

From reading the git code, looks like there is a flag to allow this feature - uploadpack.allowReachableSHA1InWant, though I'll need to check that it actually works.

Sep 5 2022, 17:02
avivey added a comment to Q15: Can you allow git fetching specific commits?.

From a quick googling, it looks like this isn't a server configuration, but most likely the commit you're looking for just isn't in the remote.
Why are you using this command? Are you using submodules for deployment?

Sep 5 2022, 14:28

Aug 31 2022

avivey awarded T15102: Implement SSL connection for database a Hungry Hippo token.
Aug 31 2022, 20:28 · Config
avivey edited projects for T15080: Intermittent DNS issues when attempting to visit we.phorge.it, added: phorge.it install; removed Phorge Upstream.
Aug 31 2022, 20:27 · phorge.it install
avivey renamed T15051: default.pem in Arcanist is out of date - maybe remove it completely? from default.pem in Arcanist is out of date to default.pem in Arcanist is out of date - maybe remove it completely?.
Aug 31 2022, 20:21 · Arcanist
avivey added a comment to T15051: default.pem in Arcanist is out of date - maybe remove it completely?.

We've picked up an updated version of the pem at https://secure.phabricator.com/D21739 - it's not the latest, but is it recent enough?

Aug 31 2022, 20:20 · Arcanist
avivey added Q13: Is anyone allowed to add a query/application to the homepage? (Answer 6).
Aug 31 2022, 20:10
avivey added a comment to T15026: Create a migration guide to move from Phabricator to Phorge.

I've written https://we.phorge.it/w/installation_and_setup/update_from_phabricator/, and I think it's basically ready for simple case (and considering we're still compatible with Phabricator in all technical aspects).

Aug 31 2022, 20:03 · Phorge
avivey added a project to T15000: Ad hoc setup tasks: phorge.it install.
Aug 31 2022, 19:58 · phorge.it install, Upstream General/Unknown
avivey added a project to T15004: Decide who has admin/commit/security access: phorge.it install.
Aug 31 2022, 19:58 · phorge.it install, Governance, Phorge
avivey added a project to T15008: Build Welcome Site: phorge.it install.
Aug 31 2022, 19:58 · phorge.it install, Upstream General/Unknown