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

Join and rename tickets_api/ and events_api/ #29

Open
bernhardreiter opened this issue Dec 3, 2020 · 4 comments
Open

Join and rename tickets_api/ and events_api/ #29

bernhardreiter opened this issue Dec 3, 2020 · 4 comments

Comments

@bernhardreiter
Copy link
Member

The advantage of having separate python modules was that a single module can be used independently,
e.g. with different access rights or from different machines.

Since the unification of stats subqueries, I expect tickets_api and events_api to e so similar
that joining them can be done while keeping the advantage. If this is done they should be renamed to
remove the "api" part, to indicate more strongly that they are not a general programming interface, but
just a backend interface for fody.

This would reduce the code duplication (part of solving #13).

@ghost
Copy link

ghost commented Dec 3, 2020

Will the events API then still be usable even when not using OTRS as well?

@bernhardreiter
Copy link
Member Author

@wagner-certat I'm unsure if I understand the question correctly. So guessing that you mean if somebody would use the endpoints, e.g. from an OTRS, would they still be available? Yes, the joining and renaming should not have an influence on the existing endpoints. You can run the modules with or without the additional intelmq-cb-mailgen tables.

So far we do not know if and how other people use the endpoints. If you know about usage, it would be cool to define it. And then we may turn the into an API at some point. (I believe an interface can be an API, if there are several using applications that are not fully dependent. fody-backend is not originally designed to be an API, just the backend for fody.)

Hope this clarifies a bit?

@ghost
Copy link

ghost commented Dec 4, 2020

Currently, the events_api/Events Web-Interface of fody can be used without using a mailgen setup (which I mis-named above as OTRS) and is useful for a wider group of IntelMQ users (but I don't have usage numbers). I would just prefer if it stays that way, but of course I can't require it.

@bernhardreiter
Copy link
Member Author

As written above:

You can run the modules with or without the additional intelmq-cb-mailgen tables.

So it would stay the way that you can use it without. :)

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

No branches or pull requests

1 participant