Skip to content

Commit 9c66946

Browse files
committed
fix: accept the mixed-case keys Pushover issues
Pushover application tokens and user keys are 30 characters from a case-sensitive alphanumeric alphabet, but both fields were validated against /^[a-z0-9]{30}$/. A key containing a single capital - which is most of them - could not be saved at all: the field stayed outlined in red and the integration could never be created. The workaround reporters arrived at was to lowercase their own key, which is not the same key and names a different account, so the integration then saved and silently delivered nothing. Both fields move together, as they share the same alphabet. Fixes #726
1 parent 1a2045b commit 9c66946

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

server/integrations/pushover.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,8 @@ export default (registerEvent) => {
2525
return {
2626
icon: "fa-solid fa-pushover",
2727
fields: [
28-
{name: "token", type: "text", required: true, regex: /^[a-z0-9]{30}$/},
29-
{name: "user_key", type: "text", required: true, regex: /^[a-z0-9]{30}$/},
28+
{name: "token", type: "text", required: true, regex: /^[A-Za-z0-9]{30}$/},
29+
{name: "user_key", type: "text", required: true, regex: /^[A-Za-z0-9]{30}$/},
3030
{name: "send_finished", type: "boolean", required: false},
3131
{name: "finished_message", type: "textarea", required: false},
3232
{name: "send_failed", type: "boolean", required: false},

0 commit comments

Comments
 (0)