Global rules contains sonyentertainmentnetwork.com twice #1471

Closed
opened 2026-02-05 01:02:16 +03:00 by OVERLORD · 1 comment
Owner

Originally created by @nielsbasjes on GitHub (Jan 16, 2023).

The global domain rules contains sonyentertainmentnetwork.com in 2 lists.
Once with sony.com and once with playstation.com

https://github.com/dani-garcia/vaultwarden/blob/main/src/static/global_domains.json#L405
https://github.com/dani-garcia/vaultwarden/blob/main/src/static/global_domains.json#L897

To me it seems that these 3 should be combined into a single list.
Is my assessment correct?

Originally created by @nielsbasjes on GitHub (Jan 16, 2023). The global domain rules contains sonyentertainmentnetwork.com in 2 lists. Once with sony.com and once with playstation.com https://github.com/dani-garcia/vaultwarden/blob/main/src/static/global_domains.json#L405 https://github.com/dani-garcia/vaultwarden/blob/main/src/static/global_domains.json#L897 To me it seems that these 3 should be combined into a single list. Is my assessment correct?
Author
Owner

@BlackDex commented on GitHub (Jan 16, 2023):

We use a tool 9b7e86efc2/tools/global_domains.py

Which extracts the data from the official Bitwarden data located here
https://github.com/bitwarden/server/blob/master/src/Core/Enums/GlobalEquivalentDomainsType.cs and https://github.com/bitwarden/server/blob/master/src/Core/Utilities/StaticStore.cs

Which states both, ans they have different main domains.
So i think it's valid. If you think otherwise, i suggest to report this at Bitwarden.

@BlackDex commented on GitHub (Jan 16, 2023): We use a tool https://github.com/dani-garcia/vaultwarden/blob/9b7e86efc23d6bbb5ee350160085ab8d8421d628/tools/global_domains.py Which extracts the data from the official Bitwarden data located here https://github.com/bitwarden/server/blob/master/src/Core/Enums/GlobalEquivalentDomainsType.cs and https://github.com/bitwarden/server/blob/master/src/Core/Utilities/StaticStore.cs Which states both, ans they have different main domains. So i think it's valid. If you think otherwise, i suggest to report this at Bitwarden.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/vaultwarden#1471