You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
58 lines
2.5 KiB
58 lines
2.5 KiB
using Bit.Core.Billing.Enums; |
|
using Bit.Core.Utilities; |
|
using Xunit; |
|
|
|
namespace Bit.Core.Test.Utilities; |
|
|
|
|
|
public class StaticStoreTests |
|
{ |
|
[Fact] |
|
public void StaticStore_Initialization_Success() |
|
{ |
|
var plans = StaticStore.Plans.ToList(); |
|
Assert.NotNull(plans); |
|
Assert.NotEmpty(plans); |
|
Assert.Equal(22, plans.Count); |
|
} |
|
|
|
[Theory] |
|
[InlineData(PlanType.EnterpriseAnnually)] |
|
[InlineData(PlanType.EnterpriseMonthly)] |
|
[InlineData(PlanType.TeamsMonthly)] |
|
[InlineData(PlanType.TeamsAnnually)] |
|
[InlineData(PlanType.TeamsStarter)] |
|
public void StaticStore_GetPlan_Success(PlanType planType) |
|
{ |
|
var plan = StaticStore.GetPlan(planType); |
|
Assert.NotNull(plan); |
|
Assert.Equal(planType, plan.Type); |
|
} |
|
|
|
[Fact] |
|
public void StaticStore_GlobalEquivalentDomains_OnlyAsciiAllowed() |
|
{ |
|
// Ref: https://daniel.haxx.se/blog/2025/05/16/detecting-malicious-unicode/ |
|
// URLs can contain unicode characters that to a computer would point to completely seperate domains but to the |
|
// naked eye look completely identical. For example 'g' and 'ց' look incredibly similar but when included in a |
|
// URL would lead you somewhere different. There is an opening for an attacker to contribute to Bitwarden with a |
|
// url update that could be missed in code review and then if they got a user to that URL Bitwarden could |
|
// consider it equivalent with a cipher in the users vault and offer autofill when we should not. |
|
// GitHub does now show a warning on non-ascii characters but it could still be missed. |
|
// https://github.blog/changelog/2025-05-01-github-now-provides-a-warning-about-hidden-unicode-text/ |
|
|
|
// To defend against this: |
|
// Loop through all equivalent domains and fail if any contain a non-ascii character |
|
// non-ascii character can make a valid URL so it's possible that in the future we have a domain |
|
// we want to allow list, that should be done through `continue`ing in the below foreach loop |
|
// only if the domain strictly equals (do NOT use InvariantCulture comparison) the one added to our allow list. |
|
foreach (var domain in StaticStore.GlobalDomains.SelectMany(p => p.Value)) |
|
{ |
|
for (var i = 0; i < domain.Length; i++) |
|
{ |
|
var character = domain[i]; |
|
Assert.True(char.IsAscii(character), $"Domain: {domain} contains non-ASCII character: '{character}' at index: {i}"); |
|
} |
|
} |
|
} |
|
}
|
|
|