forced the protocol to lowercase in AppArmorProfile

[kanhaiya04]

Purpose of PR?:

Fixes #1019
Fixed the Network policy failure

Does this PR introduce a breaking change? No

If the changes in this PR are manually verified, list down the scenarios covered::

Additional information for reviewer? : Fixes #1019

Checklist:

• Bug fix. Fixes #
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update
• PR Title follows the convention of <type>(<scope>): <subject>
• Commit has unit tests
• Commit has integration tests

[kanhaiya04]

Hey @achrefbensaad, I can't able to understand why 1 check got failed.
Is it not the right way to fix this?

[achrefbensaad]

Hi @kanhaiya04 , Thank you for your contribution.

Failures can occur due to a flaykiness of the CI.
On the other side, your current changes are CPU intensive. Please consider making the required changes in this block https://github.com/kubearmor/KubeArmor/blob/main/KubeArmor/enforcer/appArmorProfile.go#L223-L255

[kanhaiya04]

Hi @achrefbensaad, thanks for the guidance. I have made the changes accordingly, can you please review it.

[achrefbensaad]

Hi @kanhaiya04 , Thank you for your contribution.

[daemon1024]

LGTM 👌🏽 Appreciate the contribution

[daemon1024]

go-sec should pass after we merge #1098

[achrefbensaad]

Hi @kanhaiya04 , Can you please rebase your code on top of main. Ill merge after a green pipeline.

[kanhaiya04]

Hi @achrefbensaad, Done.
Do I have to create PR again from main branch?

[achrefbensaad]

No, after rebasing, please push to this PR.