CVE-2024-28175 (Other, published 2024-03-13T21:16Z)
Description: value: Argo CD is a declarative, GitOps continuous delivery tool for Kubernetes. Due to the improper URL protocols filtering of links specified in the `link.argocd.argoproj.io` annotations in the application summary component, an attacker can achieve cross-site scripting with elevated permissions. All unpatched versions of Argo CD starting with v1.0.0 are vulnerable to a cross-site scripting (XSS) bug allowing a malicious user to inject a javascript: link in the UI. When clicked by a victim user, the script will execute with the victim's permissions (up to and including admin). This vulnerability allows an attacker to perform arbitrary actions on behalf of the victim via the API, such as creating, modifying, and deleting Kubernetes resources. A patch for this vulnerability has been released in Argo CD versions v2.10.3 v2.9.8, and v2.8.12. There are no completely-safe workarounds besides upgrading. The safest alternative, if upgrading is not possible, would be to create a Kubernetes admission controller to reject any resources with an annotation starting with link.argocd.argoproj.io or reject the resource if the value use an improper URL protocol. This validation will need to be applied in all clusters managed by ArgoCD.


Severity: nan (CVSS v3 base score: )
Weakness: NVD-CWE-noinfo \u2014 Insufficient Information
There is insufficient information about the issue to classify it; details are unkown or unspecified.
Repository: https://github.com/argoproj/argo-cd
Files changed:
ui/src/app/applications/components/application-summary/application-summary.tsx
ui/src/app/applications/components/application-urls.test.ts
ui/src/app/applications/components/application-urls.tsx
Commit message: Merge pull request from GHSA-jwv5-8mqv-g387

* fix: Validate external URLs for applicatins

Signed-off-by: Ry0taK <49341894+Ry0taK@users.noreply.github.com>

* fix(ui): remove invalid external-link

Signed-off-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com>

* linting

Signed-off-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com>

---------

Signed-off-by: Ry0taK <49341894+Ry0taK@users.noreply.github.com>
Signed-off-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com>
Co-authored-by: Alexandre Gaudreault <alexandre_gaudreault@intuit.com>

Vulnerable code:

    const urls = app.status.summary.externalURLs || [];
                    {urls
                        .map(item => item.split('|'))
                        .map((parts, i) => (
                            <a key={i} href={parts.length > 1 ? parts[1] : parts[0]} target='__blank'>
                                {parts[0]} &nbsp;
                        ))}
import {ExternalLink, InvalidExternalLinkError} from './application-urls';
export const ApplicationURLs = ({urls}: {urls: string[]}) => {
        if (a.title !== '' && b.title !== '') {
        } else if (a.title === '') {
        } else if (b.title === '') {

Fixed code:
import {ExternalLinks} from '../application-urls';
    const urls = ExternalLinks(app.status.summary.externalURLs);
                    {urls.map((url, i) => {
                        return (
                            <a key={i} href={url.ref} target='__blank'>
                                {url.title} &nbsp;
                        );
                    })}
import { ExternalLink, ExternalLinks, InvalidExternalLinkError } from './application-urls';
    expect(() => {
        const _ = new ExternalLink('title|data:text/html;<h1>hi</h1>');
    }).toThrowError(InvalidExternalLinkError);
    expect(() => {
        const _ = new ExternalLink('data:title|data:text/html;<h1>hi</h1>');
    }).toThrowError(InvalidExternalLinkError);

    expect(() => {
        const _ = new ExternalLink('data:title|https://localhost:8080/applications');
    }).not.toThrowError(InvalidExternalLinkError);


test('URLs format', () => {
    expect(new ExternalLink('https://localhost:8080/applications')).toEqual({
        ref: 'https://localhost:8080/applications',
        title: 'https://localhost:8080/applications',
    })
    expect(new ExternalLink('title|https://localhost:8080/applications')).toEqual({
        ref: 'https://localhost:8080/applications',
        title: 'title',
    })
});


test('malicious URLs from list to be removed', () => {
    const urls: string[] = [
        'javascript:alert("hi")',
        'https://localhost:8080/applications',
    ]
    const links = ExternalLinks(urls);

    expect(links).toHaveLength(1);
    expect(links).toContainEqual({
        ref: 'https://localhost:8080/applications',
        title: 'https://localhost:8080/applications',
    });
});


test('list to be sorted', () => {
    const urls: string[] = [
        'https://a',
        'https://b',
        'a|https://c',
        'z|https://c',
        'x|https://d',
        'x|https://c',
    ]
    const links = ExternalLinks(urls);

    // 'a|https://c',
    // 'x|https://c',
    // 'x|https://d',
    // 'z|https://c',
    // 'https://a',
    // 'https://b',
    expect(links).toHaveLength(6);
    expect(links[0].title).toEqual('a')
    expect(links[1].title).toEqual('x')
    expect(links[1].ref).toEqual('https://c')
    expect(links[2].title).toEqual('x')
    expect(links[2].ref).toEqual('https://d')
    expect(links[3].title).toEqual('z')
    expect(links[4].title).toEqual('https://a')
    expect(links[5].title).toEqual('https://b')
});
export const ExternalLinks = (urls?: string[]) => {
        const hasTitle = (x: ExternalLink): boolean => {
            return x.title !== x.ref && x.title !== '';
        };

        if (hasTitle(a) && hasTitle(b) && a.title !== b.title) {
        } else if (hasTitle(b) && !hasTitle(a)) {
        } else if (hasTitle(a) && !hasTitle(b)) {
    return externalLinks;
};

export const ApplicationURLs = ({urls}: {urls: string[]}) => {
    const externalLinks: ExternalLink[] = ExternalLinks(urls);