Page MenuHomePhorge
Feed Advanced Search

Yesterday

Ekubischta added a comment to T15051: default.pem in Arcanist is out of date.

Adding for refs https://discourse.phabricator-community.org/t/arc-blind-trusting-domains-and-cert-errors/4980/4

Tue, Oct 26, 14:44

Mon, Oct 25

Ekubischta added a comment to T15045: Support SSL/TLS for MariaDB connections.

I would use arc diff

Mon, Oct 25, 23:01 · Config, Restricted Project
Ekubischta added a comment to T15045: Support SSL/TLS for MariaDB connections.

@Labricator Could you propose and submit a revision?

Mon, Oct 25, 13:04 · Config, Restricted Project

Thu, Oct 21

Ekubischta added a comment to D25026: Fix incorrect quoting of author in 'arc patch'.

I feel like this is a good change. I do not see any security vulnerabilities that could be introduced by this..

Thu, Oct 21, 13:39

Sat, Oct 16

Ekubischta accepted D25024: Prevent Line Numbers in Diffusion being copied as Tabs.
Sat, Oct 16, 15:26
Ekubischta added a comment to T15006: Re-brand Phorge.
In T15006#1429, @speck wrote:

I can provide more information later this weekend but I think it would help if we set up a virtual meeting with anyone interested in helping to get this done.

Yes - I could have some availability - Normally evenings (US Central Time) -

Sat, Oct 16, 02:11 · Phorge

Fri, Oct 15

Ekubischta added a comment to T15006: Re-brand Phorge.

@speck What can I help with here? - Are we waiting on upstream for anything currently?

Fri, Oct 15, 22:09 · Phorge

Thu, Oct 14

Ekubischta added a comment to D25021: Added cross-platform default fonts.

Could you describe in more detail what this change is doing? - Because I am not familiar with celerity, I did read through https://secure.phabricator.com/book/phabdev/article/celerity/ for some background.

Thu, Oct 14, 01:19

Wed, Oct 13

Ekubischta added a comment to D25024: Prevent Line Numbers in Diffusion being copied as Tabs.

@Leon95 - I think this revision will be approved with just including the webkit alternate -

Wed, Oct 13, 20:54
Ekubischta accepted D25015: Show confirmation dialog when closing a modal if form contents have been changed.
Wed, Oct 13, 20:53 · Maniphest
Ekubischta added a comment to D25015: Show confirmation dialog when closing a modal if form contents have been changed.
In D25015#826, @speck wrote:

I think that makes sense. Could you make a task to address this so we don’t lose track of it? Then let’s get this landed.

Wed, Oct 13, 20:52 · Maniphest
Ekubischta created T15053: Develop the translation strategy for strings on the client `JX` side.
Wed, Oct 13, 20:52
Ekubischta added a comment to D25023: Updated Arcanist default.pem with the most recent one from Mozilla.
In D25023#827, @speck wrote:

Because of security issues related to this I’d like to have a verification of this type of change since this inherently defines the trust used by arc.

Maybe 2 core members independently verify the these certificate changes? Is there an fast way to verify the individual certs changed here?

Wed, Oct 13, 20:48
Ekubischta added a comment to D25024: Prevent Line Numbers in Diffusion being copied as Tabs.

Things seem wildly inconsistent for this particular css string https://we.phorge.it/source/phorge/browse/master/?grep=user-select

Wed, Oct 13, 15:03
Ekubischta added a comment to D25022: Added the Jxxx Monogram to Phame Post Tab Titles.
In D25022#774, @bekay wrote:

Then maybe just show the monogram when logged in.

Wed, Oct 13, 01:49
Ekubischta added a comment to D25015: Show confirmation dialog when closing a modal if form contents have been changed.

I feel like we should move translation to a new revision, so we can move forward with this one.

Wed, Oct 13, 01:47 · Maniphest
Ekubischta added a comment to D25024: Prevent Line Numbers in Diffusion being copied as Tabs.

I tested this revision on Edge just to make sure there were no negative side effects - No issues to report.

Wed, Oct 13, 01:45

Tue, Oct 12

Ekubischta added inline comments to D25024: Prevent Line Numbers in Diffusion being copied as Tabs.
Tue, Oct 12, 22:13

Mon, Oct 11

Ekubischta added a comment to D25024: Prevent Line Numbers in Diffusion being copied as Tabs.

I am not seeing this behavior. (Using Edge)

Mon, Oct 11, 13:31

Wed, Oct 6

Ekubischta added a comment to T15052: Deal with DST_Root_CA_X3 expiry (Let's Encrypt).

This is a duplicate of T15051 and potentially solved with D25023

Wed, Oct 6, 16:06 · Arcanist

Mon, Oct 4

Ekubischta updated the task description for T15051: default.pem in Arcanist is out of date.
Mon, Oct 4, 13:49
Ekubischta updated the summary of D25023: Updated Arcanist default.pem with the most recent one from Mozilla.
Mon, Oct 4, 13:37
Ekubischta requested review of D25023: Updated Arcanist default.pem with the most recent one from Mozilla.
Mon, Oct 4, 13:34
Ekubischta added a revision to T15051: default.pem in Arcanist is out of date: D25023: Updated Arcanist default.pem with the most recent one from Mozilla.
Mon, Oct 4, 13:34
Ekubischta updated the task description for T15051: default.pem in Arcanist is out of date.
Mon, Oct 4, 13:24

Thu, Sep 30

Ekubischta updated the task description for T15051: default.pem in Arcanist is out of date.
Thu, Sep 30, 21:27
Ekubischta created T15051: default.pem in Arcanist is out of date.
Thu, Sep 30, 21:26

Sep 26 2021

Ekubischta added a comment to D25022: Added the Jxxx Monogram to Phame Post Tab Titles.
In D25022#768, @speck wrote:

Thinking about this a little more it could be intentional for these to not have monograms displayed in the page title. Tasks, Revisions, etc. are more meant to be directed to internal users of the system while Phame blog posts may also be directed to external users where a monogram might be more confusing. What do you think?

Sep 26 2021, 02:18

Sep 24 2021

Ekubischta updated the summary of D25022: Added the Jxxx Monogram to Phame Post Tab Titles.
Sep 24 2021, 18:03
Ekubischta updated the summary of D25022: Added the Jxxx Monogram to Phame Post Tab Titles.
Sep 24 2021, 18:02
Ekubischta added a revision to T15050: Add Monograms to Phame Post Titles: D25022: Added the Jxxx Monogram to Phame Post Tab Titles.
Sep 24 2021, 18:01 · Phame
Ekubischta requested review of D25022: Added the Jxxx Monogram to Phame Post Tab Titles.
Sep 24 2021, 18:01
Ekubischta created T15050: Add Monograms to Phame Post Titles.
Sep 24 2021, 17:41 · Phame

Sep 23 2021

Ekubischta added a comment to T15048: Allow awarding Tokens to individual Comments.

Referencing this "short" discussion that I remember from Discourse

Sep 23 2021, 13:36

Sep 22 2021

Ekubischta added a comment to T15047: Officially raise minimum required PHP version to 7.2.

Is it possible to pick a branching off point?

Sep 22 2021, 19:35 · Phorge
Ekubischta added a comment to T15047: Officially raise minimum required PHP version to 7.2.

Isn't Centos End of Life soon?

Sep 22 2021, 18:23 · Phorge
Ekubischta awarded T15048: Allow awarding Tokens to individual Comments a Mountain of Wealth token.
Sep 22 2021, 18:22

Sep 17 2021

Ekubischta added a comment to T15043: Automatically move tasks between columns on project boards.

The Herald rule should be project specific. If X happens then move task to Project->Column

Sep 17 2021, 15:35 · Projects

Jul 26 2021

Ekubischta added a comment to D25015: Show confirmation dialog when closing a modal if form contents have been changed.
Jul 26 2021, 22:31 · Maniphest

Jul 25 2021

Ekubischta awarded D25015: Show confirmation dialog when closing a modal if form contents have been changed a Like token.
Jul 25 2021, 02:55 · Maniphest

Jul 7 2021

Ekubischta added a comment to T15011: Build a Phorge Developer Environment.

I think both solutions work well

Jul 7 2021, 17:35 · Phorge General/Unknown

Jun 29 2021

Ekubischta closed T15021: Exclude the Aphlict package-lock.json from linting rule, a subtask of T15019: Make Aphlict a node package, as Resolved.
Jun 29 2021, 02:45 · Aphlict
Ekubischta closed T15021: Exclude the Aphlict package-lock.json from linting rule as Resolved.
Jun 29 2021, 02:45 · Aphlict

Jun 26 2021

Ekubischta added a comment to T15024: Landbot discussion(s), and generally ensuring that what is landed is what was reviewed.

There's also the point of users being used to arc land pushing code from their machine, so switching its behavior to delivering different code could have adverse UX.

Jun 26 2021, 18:40 · Phorge General/Unknown
Ekubischta added a comment to T15024: Landbot discussion(s), and generally ensuring that what is landed is what was reviewed.

diff breaks master after rebase

Jun 26 2021, 18:25 · Phorge General/Unknown
Ekubischta added a comment to D25002: T15006: Replacing external-facing trademarks/assets.

@speck A possible path forward here - We will end up with new revisions, but that is good!

Jun 26 2021, 15:59
Ekubischta closed D25012: Updated .arclint in Phorge to exclude package-lock.json from Aphlict setup.
Jun 26 2021, 15:49
Ekubischta committed rP69cb76092142: Updated .arclint in Phorge to exclude package-lock.json from Aphlict setup (authored by Ekubischta).
Updated .arclint in Phorge to exclude package-lock.json from Aphlict setup
Jun 26 2021, 15:48

Jun 25 2021

Ekubischta retitled D25012: Updated .arclint in Phorge to exclude package-lock.json from Aphlict setup from Updated .arclint in Phorge to exclude package-lock.json from Sphlict setup to Updated .arclint in Phorge to exclude package-lock.json from Aphlict setup.
Jun 25 2021, 22:29
Ekubischta added a comment to T15021: Exclude the Aphlict package-lock.json from linting rule.

D25012 solves it, but I wonder if we should just exclude it from the "type": "text" linters

Jun 25 2021, 22:27 · Aphlict
Ekubischta added a comment to D25012: Updated .arclint in Phorge to exclude package-lock.json from Aphlict setup.

We could move this from the global exclude to just the specific linters (like the txt linter, etc.) - That would be a reasonable request

Jun 25 2021, 22:26
Ekubischta requested review of D25012: Updated .arclint in Phorge to exclude package-lock.json from Aphlict setup.
Jun 25 2021, 22:25
Ekubischta added a revision to T15021: Exclude the Aphlict package-lock.json from linting rule: D25012: Updated .arclint in Phorge to exclude package-lock.json from Aphlict setup.
Jun 25 2021, 22:24 · Aphlict
Ekubischta claimed T15021: Exclude the Aphlict package-lock.json from linting rule.
Jun 25 2021, 22:18 · Aphlict
Ekubischta added a comment to T15021: Exclude the Aphlict package-lock.json from linting rule.
In T15021#564, @avivey wrote:

There might be a way to explicitly define it as generated, which (used to) exclude it from lint.

Jun 25 2021, 18:49 · Aphlict
Ekubischta added a comment to D25002: T15006: Replacing external-facing trademarks/assets.
In D25002#410, @avivey wrote:
Jun 25 2021, 17:19
Ekubischta added a comment to D25002: T15006: Replacing external-facing trademarks/assets.

Could we...

Jun 25 2021, 14:28

Jun 24 2021

Ekubischta added a comment to D25002: T15006: Replacing external-facing trademarks/assets.

What can I do to Help? -

Jun 24 2021, 23:03
Ekubischta added a comment to T15014: Develop a Phorge Release Process.

A few final thoughts here as well

Jun 24 2021, 22:41 · Phorge
Ekubischta added a comment to T15014: Develop a Phorge Release Process.

I second everything @speck says here.

Jun 24 2021, 13:13 · Phorge
Ekubischta awarded T15023: Create Phorge, a fork of Phabricator a Mountain of Wealth token.
Jun 24 2021, 03:31 · Phorge

Jun 23 2021

Ekubischta created P2 Output from avivey script.
Jun 23 2021, 22:38
Ekubischta added a comment to D25002: T15006: Replacing external-facing trademarks/assets.

Cool script @avivey! - I ran it (see P2) and see that we have a large task at hand...

Jun 23 2021, 22:35
Ekubischta raised a concern with rPe7740c8669b4: Add HarbormasterHookController as an entry point for all Harbormaster hooks.

I feel we should revert this change from master and back into a revision

Jun 23 2021, 02:57
Ekubischta requested changes to D25011: Remove CircleCI specific code from all over the place.

I have a lot of concerns about what is happening with these Harbormaster updates. I believe them to be good strategy, and should be welcomed, however..............

Jun 23 2021, 02:55

Jun 22 2021

Ekubischta added a comment to T15004: Decide who has admin/commit/security access.

T15011 discusses some of this...

Jun 22 2021, 14:16 · Phorge, Governance

Jun 21 2021

Ekubischta updated the task description for T15021: Exclude the Aphlict package-lock.json from linting rule.
Jun 21 2021, 14:28 · Aphlict
Ekubischta updated the task description for T15021: Exclude the Aphlict package-lock.json from linting rule.
Jun 21 2021, 14:27 · Aphlict
Ekubischta created T15021: Exclude the Aphlict package-lock.json from linting rule.
Jun 21 2021, 14:26 · Aphlict

Jun 20 2021

Ekubischta added a comment to T15014: Develop a Phorge Release Process.
In T15014#446, @speck wrote:

I created Release Process in our internals wiki to start the documentation on what the release process would look like, based on some of those commented. As we flesh out the plan I’d like to update that.

Jun 20 2021, 03:10 · Phorge
Ekubischta added a comment to T15016: Update default storage namespace.

One thing to consider are installs where the database is being queried outside of Phabricator - anyone doing data analytics or other tooling that interacts with the database directly would be affected by this change.

Jun 20 2021, 03:08 · Phorge General/Unknown

Jun 19 2021

Ekubischta updated the task description for T15014: Develop a Phorge Release Process.
Jun 19 2021, 21:25 · Phorge
Ekubischta added a project to T15014: Develop a Phorge Release Process: Phorge General/Unknown.
Jun 19 2021, 21:22 · Phorge
Ekubischta added a comment to D25006: Add package.json for aphlict.
Jun 19 2021, 21:21
Ekubischta created T15014: Develop a Phorge Release Process.
Jun 19 2021, 21:20 · Phorge
Ekubischta added a comment to D25006: Add package.json for aphlict.

My only concern here is that technically this introduces issues for any user install that already contains an untracked package support/aphlict/server/package-lock.json

Jun 19 2021, 18:16
Ekubischta closed D25003: Arcanist Update .arcconfig to point to we.phorge.it.
Jun 19 2021, 18:10
Ekubischta committed rARCcf4b14fcbf22: Arcanist Update .arcconfig to point to we.phorge.it (authored by Ekubischta).
Arcanist Update .arcconfig to point to we.phorge.it
Jun 19 2021, 18:10
Ekubischta added a comment to D25003: Arcanist Update .arcconfig to point to we.phorge.it.

hmm - I cannot land this revision?

Jun 19 2021, 17:04
Ekubischta added a comment to D25003: Arcanist Update .arcconfig to point to we.phorge.it.

Will need this accepted again by Blessed Committers

Jun 19 2021, 16:46
Ekubischta updated the diff for D25003: Arcanist Update .arcconfig to point to we.phorge.it.

Updating revision

Jun 19 2021, 16:46
Ekubischta added a comment to D25004: Update .gitignore to account for package-lock.json.

Without package-lock.json, it is not possible to deploy a consistent set of dependencies resolution - they might change any time any one publishes a new package, which creates a lot of problems for reproducibility.

Jun 19 2021, 15:32 · Aphlict
Ekubischta added a comment to D25002: T15006: Replacing external-facing trademarks/assets.

Test and Lint coverage?

Jun 19 2021, 15:27
Ekubischta added a comment to D25003: Arcanist Update .arcconfig to point to we.phorge.it.
In D25003#134, @speck wrote:

@Ekubischta it looks like @chris added you - could you verify your email? I'm also thinking anyone in the "security" or "blessed" groups should turn on MFA as well.

Jun 19 2021, 03:16

Jun 18 2021

Ekubischta awarded T15013: Better handling of node/npm installation for Aphlict a Baby Tequila token.
Jun 18 2021, 21:54 · Aphlict
Ekubischta added a comment to D25003: Arcanist Update .arcconfig to point to we.phorge.it.

@speck @avivey Can you add me as a Blessed Committers ??

Jun 18 2021, 20:22
Ekubischta added a comment to D25004: Update .gitignore to account for package-lock.json.

What happens if

Jun 18 2021, 20:19 · Aphlict
Ekubischta accepted D25004: Update .gitignore to account for package-lock.json.

Refs the discussion here T15011#386 and Aphlict

Jun 18 2021, 19:14 · Aphlict
Ekubischta created Aphlict.
Jun 18 2021, 19:13
Ekubischta added a comment to T15011: Build a Phorge Developer Environment.
In T15011#390, @speck wrote:

The documentation for installing Aphlict instructs you to npm install ws in the support/aphlict/server/ folder but it looks since that documentation was written newer versions of node/npm will write out package-lock.json which the repository is not setup to ignore. We'll need to add that file to the .gitignore file I think.

Jun 18 2021, 19:05 · Phorge General/Unknown
Ekubischta added a comment to D25003: Arcanist Update .arcconfig to point to we.phorge.it.
In D25003#88, @speck wrote:

This and D25001: T15006: Update .arcconfig to point to we.phorge.it are duplicates. I tried to land it this morning but ran into issues with the land process that I didn't have time to work out

Jun 18 2021, 19:01
Ekubischta added a revision to T15006: Re-brand Phorge: D25003: Arcanist Update .arcconfig to point to we.phorge.it.
Jun 18 2021, 18:01 · Phorge
Ekubischta updated the summary of D25003: Arcanist Update .arcconfig to point to we.phorge.it.
Jun 18 2021, 18:01
Ekubischta updated the summary of D25003: Arcanist Update .arcconfig to point to we.phorge.it.
Jun 18 2021, 18:01
Ekubischta added reviewers for D25003: Arcanist Update .arcconfig to point to we.phorge.it: speck, avivey, tobiaswiese.
Jun 18 2021, 18:00
Ekubischta requested review of D25003: Arcanist Update .arcconfig to point to we.phorge.it.
Jun 18 2021, 17:59
Ekubischta added a comment to T15011: Build a Phorge Developer Environment.

A few things @willson556

Jun 18 2021, 16:58 · Phorge General/Unknown
Ekubischta added a comment to T15011: Build a Phorge Developer Environment.

I actually started on a VSCode Devcontainer based solution on my GitHub: https://github.com/willson556/phorge-devcontainer

It is working pretty well with notifications and repository hosting both configured out of the box. My only concern with the config at the moment is that it's very much setup for development -- we would want to clearly document that it is not to be used as a starting point for a production docker-compose setup!

Any feedback would be appreciated!

Jun 18 2021, 16:27 · Phorge General/Unknown
Ekubischta added a comment to T15011: Build a Phorge Developer Environment.
In T15011#363, @speck wrote:

We should consider a Vagrantfile in place of docker containers. I think it will be more approachable to newcomers having a single VM with all the services/configurations setup compared to managing multiple containers.

Separately, developing on Windows has its own complications

Jun 18 2021, 15:49 · Phorge General/Unknown
chris awarded T15011: Build a Phorge Developer Environment a 100 token.
Jun 18 2021, 14:56 · Phorge General/Unknown