Paths

Table of Contentst

Differential D25903

directly
Needs ReviewPublic
Actions

Authored by aklapper on Tue, Mar 4, 17:31.

Details

Reviewers

None

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T16006: "(Exception) No such token ""!" when directly accessing token giving URI

Summary

Handle bots doing strange things more gracefully by not throwing a noisy uninteresting exception in my error log.

EXCEPTION: (Exception) No such token ""! at [<phorge>/src/applications/tokens/editor/PhabricatorTokenGivenEditor.php:153]

Closes T16006

Test Plan

Go to /token/give/PHID-TASK-someValidId/ and click one of the tokens, compare Phorge behavior before and after patch
Go to a task, in the sidebar, click "Award a Token" and "Rescind a Token": Still works as before

Diff Detail

Repository

rP Phorge

Branch

T16006TokenExcpt (branched from master)

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 1764
Build 1764: arc lint + arc unit

Event Timeline

aklapper created this revision.Tue, Mar 4, 17:31

Owners added a reviewer: O1: Blessed Committers.Tue, Mar 4, 17:31

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptTue, Mar 4, 17:31

aklapper requested review of this revision.Tue, Mar 4, 17:31

Harbormaster completed remote builds in B1764: Diff 2723.Tue, Mar 4, 17:31

aklapper edited the summary of this revision. (Show Details)Tue, Mar 4, 17:32

Per comments in T16006 this is rather workaround (ignore) instead of a fix (make it work).

Revision Contents
Changeset List

Path

Size

Coverage (All)

Coverage (Touched)

Packages

src/

applications/

tokens/

controller/

PhabricatorTokenGiveController.php

6 lines

O1: Blessed Committers

Diff 2723

View Options

src/applications/tokens/controller/PhabricatorTokenGiveController.php

Avoid "No such token" exception going to /token/give/ directlyNeeds ReviewPublicActions