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

FW API: ajout endpoints + tri #75

Merged
merged 4 commits into from
Nov 11, 2020
Merged

FW API: ajout endpoints + tri #75

merged 4 commits into from
Nov 11, 2020

Conversation

SNoof85
Copy link
Member

@SNoof85 SNoof85 commented Nov 4, 2020

Changements :

  • ⚠️ BREAKING get_port_forwarding_configuration ➡️ sert maintenant à récup la config d'une règle précise selon redir_id
  • ⚠️ BREAKING get_all_port_forwarding_configuration ➡️ était nommée avant get_port_forwarding_configuration (récup de toutes les redirections)
  • ⚠️ BREAKING get_incoming_ports_configuration ➡️ sert maintenant à récup la config d'un port entrant ouvert précis selon port_id
  • ⚠️ BREAKING get_all_incoming_ports_configuration ➡️ était nommée avant get_incoming_ports_configuration (récup de tous les ports ouverts en entrée)
  • edit_port_forwarding_configuration ➡️ Edition d'une règle précise selon redir_id

Les breakings ne le sont pas dans HA, aucun de ces appels n'est utilisé ! Uniquement dans le cas d'un usage autre de la lib.

Tri des fonctions pour une meilleure lisibilité dans cet ordre si tous présents pour les 3 catégories (redirection, incoming, dmz) :

  • create
  • delete
  • get
  • get_all
  • edit
  • set

@SNoof85 SNoof85 added breaking Breaking Changes enhancement New feature or request labels Nov 4, 2020
Copy link
Member

@Quentame Quentame left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super !


Test : ❌
Conforme à la doc : ✔️

@Quentame
Copy link
Member

Quentame commented Nov 4, 2020

CI à corriger avant de merge

@SNoof85
Copy link
Member Author

SNoof85 commented Nov 5, 2020

Par contre le fail du test je le comprends pas !

@SNoof85
Copy link
Member Author

SNoof85 commented Nov 5, 2020

Par contre le fail du test je le comprends pas !

En fait si :D

@SNoof85 SNoof85 requested a review from Quentame November 5, 2020 11:18
@Quentame Quentame changed the title FW API ajout + tri FW API: ajout endpoints + tri Nov 11, 2020
@Quentame Quentame merged commit b17a159 into hacf-fr:master Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Breaking Changes enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants