Skip to content

LLT-6647: Use libfirewall.so #1566

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.

Sign up for GitHub

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

Jump to bottom
Draft
stalowyjez wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

LLT-6647: Use libfirewall.so #1566

stalowyjez wants to merge 12 commits into from
+1,603 −6,123

Conversation

@stalowyjez
Copy link
Contributor

@stalowyjez stalowyjez commented Nov 6, 2025 •
edited
Loading

Problem

Libtelio should use dynamically loaded libfirewall.so library

Solution

Add loader, load firewall library and use it instead of the implementation inside telio-firewall, which this PR removes.

☑️ Definition of Done checklist

  • Commit history is clean (requirements)
  • README.md is updated
  • Functionality is covered by unit or integration tests

@stalowyjez stalowyjez requested a review from a team as a code owner November 6, 2025 16:17
@stalowyjez stalowyjez marked this pull request as draft November 6, 2025 16:18
@stalowyjez stalowyjez force-pushed the LLT-6647-use_libfirewall branch from 73ad664 to a1dd30e Compare November 6, 2025 16:35
@stalowyjez stalowyjez force-pushed the LLT-6647-use_libfirewall branch from e84422e to 3449633 Compare December 8, 2025 15:52
@stalowyjez stalowyjez force-pushed the LLT-6647-use_libfirewall branch from 3449633 to e97d8d9 Compare December 8, 2025 16:02
@stalowyjez stalowyjez force-pushed the LLT-6647-use_libfirewall branch from a9c4f30 to 9e04ce8 Compare December 9, 2025 11:32
@stalowyjez stalowyjez force-pushed the LLT-6647-use_libfirewall branch from 2e519e4 to ea4dbac Compare December 9, 2025 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Jauler Jauler Awaiting requested review from Jauler Jauler will be requested when the pull request is marked ready for review Jauler is a code owner

@jjanowsk jjanowsk Awaiting requested review from jjanowsk jjanowsk will be requested when the pull request is marked ready for review jjanowsk is a code owner

@tomaszklak tomaszklak Awaiting requested review from tomaszklak tomaszklak will be requested when the pull request is marked ready for review tomaszklak is a code owner

@LukasPukenis LukasPukenis Awaiting requested review from LukasPukenis LukasPukenis will be requested when the pull request is marked ready for review LukasPukenis is a code owner

@matislovas matislovas Awaiting requested review from matislovas matislovas will be requested when the pull request is marked ready for review matislovas is a code owner

@mathiaspeters mathiaspeters Awaiting requested review from mathiaspeters mathiaspeters will be requested when the pull request is marked ready for review mathiaspeters is a code owner

@olekoliinyk olekoliinyk Awaiting requested review from olekoliinyk olekoliinyk will be requested when the pull request is marked ready for review olekoliinyk is a code owner

@asledzinskiy asledzinskiy Awaiting requested review from asledzinskiy asledzinskiy will be requested when the pull request is marked ready for review asledzinskiy is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@stalowyjez