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

Conditionally add interface proxies for taint persistence #20

Merged
merged 1 commit into from
Feb 24, 2024

Conversation

leeN
Copy link
Collaborator

@leeN leeN commented Feb 23, 2024

If we always load the proxies and do not use the taint persistence driver, which might be the sensible path due to
performance/compatibility restrictions the application will break.

As the focus is not purely on GDPR tainting anymore, the addition of the problematic proxies is hidden behind a flag has has to be passed to the agent.

It would be nicer to dynamically detect this, but this a) might be slow and b) is fairly complex.

If we always load the proxies and do not use the taint persistence
driver, which might be the sensible path due to
performance/compatibility restrictions the application will break.

As the focus is not purely on GDPR tainting anymore, the addition of the
problematic proxies is hidden behind a flag has has to be passed to the
agent.

It would be nicer to dynamically detect this, but this a) might be slow
and b) is fairly complex.
@leeN leeN requested a review from tmbrbr February 23, 2024 15:20
@leeN leeN added enhancement New feature or request gdpr tainting Issues related to enforcing GDPR properties Usability How can we make Fontus easier to use? labels Feb 23, 2024
@tmbrbr tmbrbr merged commit 5d8cf63 into SAP:main Feb 24, 2024
3 of 4 checks passed
@leeN leeN deleted the proxy_config branch February 24, 2024 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request gdpr tainting Issues related to enforcing GDPR properties Usability How can we make Fontus easier to use?
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants