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

Defined observer zones and interfaces #740

Closed
wants to merge 2 commits into from
Closed

Defined observer zones and interfaces #740

wants to merge 2 commits into from

Conversation

dainok
Copy link

@dainok dainok commented Feb 4, 2020

Hi,

referring to network flows passing through a firewall, I think we should track ingress and egress security zones and interfaces.
I think we can have two different approaches:

  • zones/interfaces are defined on the observer schema, and I think that's more correct
  • zones/interfaces are defined on the client/server/source/destination schema
    In the latter case, zones/interfaces are still references observer (i.e. firewall) objects.
    Any thoughts?

@webmat
Copy link
Contributor

webmat commented Feb 4, 2020

Ping @dainperkins to align with #688 and #689

@dainperkins
Copy link
Contributor

I was thinking about just this thing over the last couple of days. I'm going to add some color on the vlan and interface PRs to get the group on board, and then we can look at the observer implementation

@dainperkins dainperkins mentioned this pull request Feb 4, 2020
@webmat
Copy link
Contributor

webmat commented Feb 4, 2020

Also I forgot something important, when I answered earlier: thanks @dainok for opening this. You're welcome to join the discussion on the other PRs as well :-)

@webmat
Copy link
Contributor

webmat commented Feb 4, 2020

@elasticmachine, run elasticsearch-ci/docs

Preview

@dainok
Copy link
Author

dainok commented Feb 15, 2020

According to #688 I moved zone under client/source/destination/server.

@dainperkins
Copy link
Contributor

after talking with @webmat I've put a new PR with the basic interface & vlan info, as well as observer based zone info for ingress/egress traffic - #752

@dainok
Copy link
Author

dainok commented Feb 20, 2020

Agree. Closing because incorporated under #752

@dainok dainok closed this Feb 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants