fix: allow non-members to open suppress modal to see details.

Author: rbjornstad

src/lib/components/image/ImageVulnerabilities.svelte

@@ -175,20 +175,20 @@
 			{#each vulnz as v (v)}
 				<Tr>
 					<Td>
-						{#if authorized}
-							<Button
-								variant="tertiary"
-								size="xsmall"
-								onclick={() => {
-									findingToSuppress = v;
-									suppressOpen = true;
-								}}
-							>
-								<code>{v.identifier}</code>
-							</Button>
-						{:else}
+						<!--{#if authorized}-->
+						<Button
+							variant="tertiary"
+							size="xsmall"
+							onclick={() => {
+								findingToSuppress = v;
+								suppressOpen = true;
+							}}
+						>
 							<code>{v.identifier}</code>
-						{/if}
+						</Button>
+						<!--{:else}
+							<code>{v.identifier}</code>
+						{/if}-->
 					</Td>
 					<Td><code>{v.package}</code></Td>
 					<Td
@@ -236,7 +236,7 @@
 		}}
 	/>
 
-	{#if findingToSuppress && authorized !== PendingValue && authorized && image.workloadReferences}
+	{#if findingToSuppress && authorized !== PendingValue && image.workloadReferences}
 		{#key findingToSuppress.id}
 			<SuppressFinding
 				bind:open={suppressOpen}

src/lib/components/image/SuppressFinding.svelte

@@ -226,39 +226,44 @@
 			</Tbody>
 		</Table>
 	</div>
+	{#if authorized}
+		<div class="wrapper">
+			{#if errormessage !== ''}
+				<Alert variant="error">
+					{errormessage}
+				</Alert>
+			{/if}
+			<p>
+				Please provide a reason for suppressing this finding. This will be recorded in the analysis
+				audit log. Suppression will be in effect for all workloads using this image.
+			</p>
+			<Select size="small" label="Analysis" bind:value={selectedReason}>
+				{#each SUPPRESS_OPTIONS as option (option)}
+					{#if option.value === finding.state}
+						<option value={option.value}>{option.text} </option>
+					{:else}
+						<option value={option.value}>{option.text}</option>
+					{/if}
+				{/each}
+			</Select>
 
-	<div class="wrapper">
-		{#if errormessage !== ''}
-			<Alert variant="error">
-				{errormessage}
-			</Alert>
-		{/if}
-		<p>
-			Please provide a reason for suppressing this finding. This will be recorded in the analysis
-			audit log. Suppression will be in effect for all workloads using this image.
-		</p>
-		<Select size="small" label="Analysis" bind:value={selectedReason}>
-			{#each SUPPRESS_OPTIONS as option (option)}
-				{#if option.value === finding.state}
-					<option value={option.value}>{option.text} </option>
-				{:else}
-					<option value={option.value}>{option.text}</option>
-				{/if}
-			{/each}
-		</Select>
-
-		<TextField type="text" bind:value={inputText}>
-			{#snippet label()}
-				Comment
-			{/snippet}
-		</TextField>
-		<Checkbox bind:checked={suppressed}>Suppress</Checkbox>
-	</div>
+			<TextField type="text" bind:value={inputText}>
+				{#snippet label()}
+					Comment
+				{/snippet}
+			</TextField>
+			<Checkbox bind:checked={suppressed}>Suppress</Checkbox>
+		</div>
+	{/if}
 	{#snippet footer()}
-		<Button variant="primary" size="small" onclick={triggerSuppress} disabled={!authorized}
-			>Update</Button
-		>
-		<Button variant="secondary" size="small" onclick={close}>Cancel</Button>
+		{#if authorized}
+			<Button variant="primary" size="small" onclick={triggerSuppress} disabled={!authorized}
+				>Update</Button
+			>
+			<Button variant="secondary" size="small" onclick={close}>Cancel</Button>
+		{:else}
+			<Button variant="secondary" size="small" onclick={close}>Close</Button>
+		{/if}
 	{/snippet}
 </Modal>