starred/pocket-id-pocket-id-1
1
0
Fork 0
mirror of https://github.com/pocket-id/pocket-id.git synced 2025-12-11 07:42:58 +03:00
Code Issues 122 Packages Projects Releases 10 Wiki Activity

🐛 Bug Report: SMTP field type conversion error when starting with a number #338

New Issue
Closed
opened 2025-10-08 00:04:07 +03:00 by OVERLORD · 3 comments
OVERLORD commented 2025-10-08 00:04:07 +03:00
Owner
Copy Link

Originally created by @simkli on GitHub.

Reproduction steps

When I set a SMTP username or password which starts with a number all letters are being removed.

Reproduction:

  1. Go to Admin Panel > Application Settings
  2. Enter "1234ABC" as SMTP password or username
  3. Save it
  4. Reload page (Important for reproduction!!)
  5. Only 1234 is being saved

Expected behavior

Always save the fields as a string

Actual Behavior

If the field value starts with a number the value is converted to a number.

Version and Environment

v0.34.0

Log Output

No response

Originally created by @simkli on GitHub. ### Reproduction steps When I set a SMTP username or password which starts with a number all letters are being removed. Reproduction: 1. Go to Admin Panel > Application Settings 2. Enter "1234ABC" as SMTP password or username 3. Save it 4. Reload page (**Important for reproduction!!**) 5. Only 1234 is being saved ### Expected behavior Always save the fields as a string ### Actual Behavior If the field value starts with a number the value is converted to a number. ### Version and Environment v0.34.0 ### Log Output _No response_
OVERLORD added the bug label 2025-10-08 00:04:07 +03:00
OVERLORD commented 2025-10-08 00:04:09 +03:00
Author
Owner
Copy Link

@simkli commented on GitHub:

Thank you so much ❤️

@simkli commented on GitHub: Thank you so much ❤️
OVERLORD commented 2025-10-08 00:04:10 +03:00
Author
Owner
Copy Link

@stonith404 commented on GitHub:

Thanks this should be fixed in v0.35.0.

@stonith404 commented on GitHub: Thanks this should be fixed in `v0.35.0`.
OVERLORD commented 2025-10-08 00:04:10 +03:00
Author
Owner
Copy Link

@simkli commented on GitHub:

Just checked the database. Seems to be correctly stored there. Maybe it's just an UI thing, but it prevents me to make changes without copying in the correct credentials again.

@simkli commented on GitHub: Just checked the database. Seems to be correctly stored there. Maybe it's just an UI thing, but it prevents me to make changes without copying in the correct credentials again.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
breaking
bug
documentation
feature
needs more upvotes
open to pull requests
pull-request
Mirrored from GitHub Pull Request
No Label bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/pocket-id-pocket-id-1#338
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?