Fixed auth type non-auth for IKE profile

[barloff-st]

Description

IKE profile takes non-auth and not none as an authentication hash type.

TASK [palo_alto : Create Secure IKE Crypto Profile] ***************************************************************************************
fatal: [TEST_PaloAlto.hostname]: FAILED! => {"changed": false, "msg": "Failed apply:  TEST-IKE-Crypto-Profile -> hash 'none' is not an allowed keyword\n TEST-IKE-Crypto-Profile -> hash is invalid"}

Motivation and Context

aes-gcm-256 encryption has built in authentication, so you need to set the authentication to non-auth.

How Has This Been Tested?

Tested running the following on Rocky 8:

- name: Create Secure IKE Crypto Profile
  paloaltonetworks.panos.panos_ike_crypto_profile:
  ip_address: "{{ firewall_ip }}" #TEMP FOR SCRIPT TESTING, Used for ansible connection to PA
  username: "{{ username }}"
  password: "{{ password }}"
  state: 'present'
  name: 'TEST-IKE-Crypto-Profile'
  dh_group: ['group20']
  #authentication: 'none' #BUG: Authenication will leave at default "sha1" when using "none"
  authentication: 'non-auth'
  encryption: ['aes-256-gcm']
  lifetime_seconds: '28800'

Types of changes

• Bug fix (non-breaking change which fixes an issue)

Checklist

• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes if appropriate.
• All new and existing tests passed.

[welcome-to-palo-alto-networks]

🎉 Thanks for opening this pull request! We really appreciate contributors like you! 🙌

[barloff-st]

I do not believe none to be a valid option, but if it is in some older version of Pan OS, I can change this to simply add non-auth to the option list instead of replacing none.

[barloff-st]

See bug #419

[shinmog]

lgtm

[welcome-to-palo-alto-networks]

🎉 Congrats on getting your first pull request merged! We here at Palo Alto Networks are so grateful! ❤️

[github-actions]

🎉 This PR is included in version 2.17.2 🎉

The release is available on Ansible Galaxy and GitHub release

Posted by semantic-release bot