Change favicon/company enrichment urls (#9065)
We're moving favicon/telemetry/company enrichment to a separate url for better security/monitoring
This commit is contained in:
@ -17,27 +17,27 @@ describe('sanitizeURL', () => {
|
||||
describe('getLogoUrlFromDomainName', () => {
|
||||
test('should return the correct logo URL for a given domain', () => {
|
||||
expect(getLogoUrlFromDomainName('example.com')).toBe(
|
||||
'https://favicon.twenty.com/example.com',
|
||||
'https://twenty-icons.com/example.com',
|
||||
);
|
||||
|
||||
expect(getLogoUrlFromDomainName('http://example.com/')).toBe(
|
||||
'https://favicon.twenty.com/example.com',
|
||||
'https://twenty-icons.com/example.com',
|
||||
);
|
||||
|
||||
expect(getLogoUrlFromDomainName('https://www.example.com/')).toBe(
|
||||
'https://favicon.twenty.com/example.com',
|
||||
'https://twenty-icons.com/example.com',
|
||||
);
|
||||
|
||||
expect(getLogoUrlFromDomainName('www.example.com')).toBe(
|
||||
'https://favicon.twenty.com/example.com',
|
||||
'https://twenty-icons.com/example.com',
|
||||
);
|
||||
|
||||
expect(getLogoUrlFromDomainName('example.com/')).toBe(
|
||||
'https://favicon.twenty.com/example.com',
|
||||
'https://twenty-icons.com/example.com',
|
||||
);
|
||||
|
||||
expect(getLogoUrlFromDomainName('apple.com')).toBe(
|
||||
'https://favicon.twenty.com/apple.com',
|
||||
'https://twenty-icons.com/apple.com',
|
||||
);
|
||||
});
|
||||
|
||||
|
||||
@ -9,6 +9,6 @@ export const getLogoUrlFromDomainName = (
|
||||
): string | undefined => {
|
||||
const sanitizedDomain = sanitizeURL(domainName);
|
||||
return sanitizedDomain
|
||||
? `https://favicon.twenty.com/${sanitizedDomain}`
|
||||
? `https://twenty-icons.com/${sanitizedDomain}`
|
||||
: undefined;
|
||||
};
|
||||
|
||||
Reference in New Issue
Block a user