Page MenuHomePhabricator
Create Task
Maniphest T10090

Updating Herald rule throws Unhandled Exception
Closed, ResolvedPublic
Actions

Description

  1. Created Global rule for Commits.
  2. Add field: "is_merge_commit" false.
  3. Hit save rule.
  4. Hit edit and tried to add another rule: commit message doesn't contain some text.
  5. Hit save rule.

Outcome:
Unhanded Exception ("PhutilProxyException")
Failed to decode rule data

Related Objects

Event Timeline

• alnet82 created this task.Jan 6 2016, 3:36 PM
chad added a subscriber: chad.Jan 6 2016, 4:15 PM

I can't reproduce this at HEAD. Are there more complete instructions we can follow?

chad added a comment.Jan 6 2016, 4:17 PM

What I created to test:

pasted_file (503×362 px, 52 KB)

joshuaspence added a subscriber: joshuaspence.Jan 7 2016, 7:59 AM
• alnet82 added a comment.Jan 7 2016, 1:12 PM

I upgraded a 6 month old installation to HEAD. Can it be related? Didn't experience any other issues.

• alnet82 updated the task description. (Show Details)Jan 10 2016, 7:31 AM
clemvangelis added a subscriber: clemvangelis.EditedJan 18 2016, 3:26 PM

I'm hitting the same problem using Chrome (Version 41.0.2272.89 (64-bit)).
However when using firefox, I do not have this problem so it seems to be browser related.

@chad , Is there any way we can get more data to help debugging this problem ?

By the way, i'm using the following version of phabricator

phab : https://secure.phabricator.com/rPf59ebf4c09598e5f02657a4037c822b64a306646

libphutil: https://secure.phabricator.com/rPHU0fa7efbf09d75b7803ef54bd6c72ee3c69a964e4

arcanist: https://secure.phabricator.com/rARCaeb374b3334891d436334ac382bb63e2e4620c95

By the way, I can confirm this only happens when editing a rule. Creating one works fine.

• alnet82 mentioned this in T10181: Create audit requests based on a package commits.Jan 20 2016, 7:35 AM
epriestley closed this task as Resolved.May 10 2016, 4:20 PM
epriestley claimed this task.
epriestley added a subscriber: epriestley.

This may have been fixed by D15507. Otherwise, we can't reproduce it, so we can't move forward. See Providing Reproduction Steps.

Herald added a subscriber: eadler. · View Herald TranscriptMay 10 2016, 4:20 PM
clemvangelis added a comment.May 11 2016, 7:07 AM

Indeed, it's fixed ! Thanks

Phabricator · Built by Phacility