Skip to content

Create Fortinet-Fortigate-Traffic-parser#4542

Closed
stuartjordan wants to merge 1 commit into
Azure:masterfrom
stuartjordan:patch-1
Closed

Create Fortinet-Fortigate-Traffic-parser#4542
stuartjordan wants to merge 1 commit into
Azure:masterfrom
stuartjordan:patch-1

Conversation

@stuartjordan
Copy link
Copy Markdown

Required items, please complete

Change(s):

  • New file - function to extract fields from SyslogMessage field in Fortinet Fortigate syslog messages into new columns.

Reason for Change(s):

  • New function - not present on the repo

Version Updated:

  • This is version 1.0

Testing Completed:

  • Ran against my own data

Checked that the validations are passing and have addressed any issues that are present:

  • No issues

Extracts fields from SyslogMessage field into new columns.
@ghost
Copy link
Copy Markdown

ghost commented Mar 31, 2022

CLA assistant check
All CLA requirements met.

@v-sabiraj v-sabiraj self-assigned this Mar 31, 2022
@v-sabiraj v-sabiraj added the Parser Parser specialty review needed label Mar 31, 2022
@v-sabiraj
Copy link
Copy Markdown
Contributor

Hey @stuartjordan, can you please move the parser to solution folders of Fortinet-FortiGate, please check and update, thanks.

@iionescu
Copy link
Copy Markdown

iionescu commented Apr 5, 2022

Hey folks, when do you think you'll have a chance to review this, please?

@v-sabiraj
Copy link
Copy Markdown
Contributor

Hey @iionescu , there is a comment already added to move the parser to the Solution folder, it is directing to the master, please check and update, thanks.

@v-sabiraj
Copy link
Copy Markdown
Contributor

Hey @stuartjordan, can you please check and move the file to Parsers folder or the Solutions folder of Fortinet-Fortigate. Thanks.

@v-sabiraj
Copy link
Copy Markdown
Contributor

Hey @stuartjordan, can you please check and move the file to Parsers folder or the Solutions folder of Fortinet-Fortigate so, we can proceed to merge, Thanks.

@v-marimanda v-marimanda added the Waiting on poster Waiting for reply from original person who posted the issue label Apr 19, 2022
@v-sabiraj
Copy link
Copy Markdown
Contributor

Hey @stuartjordan @iionescu, can you please check and move the file to Parsers folder or the Solutions folder of Fortinet-Fortigate so, we can proceed to merge, Thanks.

@v-sabiraj
Copy link
Copy Markdown
Contributor

Hey @stuartjordan, please move the file to Parsers folder or the Solutions folder of Fortinet-Fortigate.

@v-sabiraj
Copy link
Copy Markdown
Contributor

Hey @stuartjordan, will you please make the requested change, thanks.

@v-dvedak
Copy link
Copy Markdown
Collaborator

Hey @stuartjordan , we wanted to check on the status of this PR Create Fortinet-Fortigate-Traffic-parser. It is pending from more than 60 days. Let us know if any assistance is required for this PR. Per our standard operating procedures if no response is received in the next 7 business days we will close this PR. Thank you for your cooperation.

@v-dvedak
Copy link
Copy Markdown
Collaborator

Hi @stuartjordan , Since we have not received response from last 7 days, we are closing your PR per our standard operating procedures. If you still need support for this issue you can re-open the PR at any time.
If you do re-open, we simply request that you ensure the PR has response to the last request. Thank you for your cooperation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Parser Parser specialty review needed Waiting on poster Waiting for reply from original person who posted the issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants