Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upload processing - Virus Scan #163

Closed
5 tasks done
Tracked by #84
RagnarFatland-Avanade opened this issue Nov 7, 2023 · 5 comments
Closed
5 tasks done
Tracked by #84

Upload processing - Virus Scan #163

RagnarFatland-Avanade opened this issue Nov 7, 2023 · 5 comments
Assignees
Labels
kind/user-story Used for issues that describes functionality for our users. product/formidling Issues related til Altinn Broker

Comments

@RagnarFatland-Avanade
Copy link
Collaborator

RagnarFatland-Avanade commented Nov 7, 2023

UploadProcessing for uploaded files with Virus Scanning.
Upload file triggers async message to Antimalware solution.
Triggers virus scan of file.
Result pushed to Broker component API to update status for file.
Future: Publishes Altinn Event for Upload Processing Result.

User Stories

Preview Give feedback
  1. kind/user-story
    tomshag
  2. kind/user-story
  3. kind/chore kind/feature product/formidling
    Ceredron tomshag
  4. product/formidling
    tomshag
  5. kind/user-story
    Ceredron
@leogasnier
Copy link

Innspill i åpent møte:
[12:25 PM] Jendal, Håkon
Tenker at ende til ende kryptering bør være beste praksis for slike type løsninger. Tilsvarer da slik man gjør det i Norsk helsenett for for eksempel Dialogmeldinger i helsevesenet.

Videre så kommenteres det at virusscan bør skje på mottakersiden.

Vi må vurdere hva vi krever og hva som skal være opsjoner satt i konfigurasjon av tjenesten, eller instans.

@RagnarFatland-Avanade
Copy link
Collaborator Author

RagnarFatland-Avanade commented Nov 9, 2023

Jeg vil si at eventuell opsjon om virusscan må settes på tjenesten, ikke instansnivå. Siden det ellers vil være avsender som styrer både potensielt infisert innhold og om det scannes.
Mest fleksible/komplekse variant hadde vært en opsjon på tjenestenivå med; "Aldri", "Styrt av Avsender", "Alltid".
Det bør i så fall også være tilsvarende opsjoner for CheckSum-validering.

@leogasnier leogasnier added kind/feature A distinct functionality or enhancement added to a product to provide specific value to users product/formidling Issues related til Altinn Broker labels Nov 13, 2023
@leogasnier leogasnier moved this from 🆕 New to 📋 Backlog in Altinn melding og formidling Nov 13, 2023
@RagnarFatland-Avanade RagnarFatland-Avanade moved this from 📋 Backlog to 🏗 In progress in Altinn melding og formidling Nov 20, 2023
@hansdragnes hansdragnes added this to the MVP milestone Nov 22, 2023
@RagnarFatland-Avanade
Copy link
Collaborator Author

Etter møte 21.11.2023 sammen med personer fra Altinn Platform / Notifications, så kom vi frem til at Kafka ikke er aktuelt i denne sammenheng. Internt referat her.

Vi går videre med å analysere ny funksjonalitet i Microsoft Defender for Azure Storage for å se om denne er aktuell som basis.
Eventuelt å benytte/utvide Altinn File-Scan.

@tomshag
Copy link
Contributor

tomshag commented Dec 1, 2023

  1. Pricing information: https://azure.microsoft.com/en-us/pricing/details/defender-for-cloud/

  2. Scanning er automatisk når den er satt opp. (https://learn.microsoft.com/en-us/azure/defender-for-cloud/defender-for-storage-malware-scan#set-up-of-malware-scanning)
          a. Er det mulig å sette opp defender til å ikke automatisk scanne, mne istedet scanne basert på "bestilling".
                Svar: fra det jeg har funnet finnes ingen slik mulighet.
                Blobs blir scannet hver gang de har blitt "endret" (on-upload trigger). (https://learn.microsoft.com/en-us/answers/questions/1374275/how-to-trigger-an-on-demand-style-antivirus-scan-w)
                Kommentar: "All upload methods trigger the scan. Modifying a blob is an upload operation and therefore the modified content is scanned after the update."
          b. Det er mulig å sette defender basert på storage account, istedet for subscription. Det burde altså være mulig å la tjenesteeiere ha brokerservice hvor scanning ikke forekommer. Om det er ønskelig å ha brokerservice hvor kun spesifikke filer blir scannet ser ikke dette ut til å være tilgjengelig med Defender for øyeblikket.
    Oppsett av defender har spesifikke krav ihht. roller for Azure subscription:
    https://learn.microsoft.com/en-us/azure/defender-for-cloud/support-matrix-defender-for-storage

      
3. Sjekking av resultat kan gjøres via flere metoder.
      a. Event grid, for eksempel hentet via webhook.
            https://learn.microsoft.com/en-us/azure/event-grid/webhook-event-delivery
            https://learn.microsoft.com/en-us/azure/event-grid/custom-event-to-function
      b. Blob storage tag, hentes fra Blob storage.
            https://learn.microsoft.com/en-us/azure/storage/blobs/storage-blob-index-how-to

@hansdragnes hansdragnes moved this from 🏗 In progress to 👀 In review in Altinn melding og formidling Dec 1, 2023
@hansdragnes hansdragnes moved this from 👀 In review to 🏗 In progress in Altinn melding og formidling Dec 1, 2023
@tomshag
Copy link
Contributor

tomshag commented Jan 9, 2024

Maximum filstørrelse som scannes i Azure Defender: 2GB.

Maximum filstørrelse som scannes via Symantec Protection Engine: 30GB
(muligens finnes det ingen max grense:
https://techdocs.broadcom.com/us/en/symantec-security-software/endpoint-security-and-management/symantec-protection-engine/9-1-0/Configuring-and-Monitoring/specifying-the-maximum-file-or-message-size-to-sca-v128532379-d4995e12564.html#v128532379)

@hansdragnes hansdragnes removed this from the MVP milestone Jan 12, 2024
@Ceredron Ceredron moved this from 🏗 In progress to ✅ Done in Altinn melding og formidling Feb 6, 2024
@Ceredron Ceredron closed this as completed Mar 4, 2024
@leogasnier leogasnier added kind/user-story Used for issues that describes functionality for our users. and removed kind/feature A distinct functionality or enhancement added to a product to provide specific value to users labels Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/user-story Used for issues that describes functionality for our users. product/formidling Issues related til Altinn Broker
Projects
Status: ✅ Done
Development

No branches or pull requests

5 participants