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

Remove authmanager and webconfig #456

Merged
merged 7 commits into from
Nov 17, 2021

Conversation

Lagovas
Copy link
Collaborator

@Lagovas Lagovas commented Nov 12, 2021

We decided to remove webconfig and authmanager. So here:

  • updated dockerfiles
  • removed from keystore interfaces
  • removed cmd binary packages
  • removed from tests
  • removed from Makefile and related scripts
  • removed from acra-keymaker and acra-keys generation symmetric key for basic auth credentials
  • removed endpoints from acra-server HTTP API related to webconfig that provided API to download/upload config of acra-server (subset of config actually of 5 params)

Draft until I run and check all dockerfiles and be sure that all of them work. All other staff is ready to review.

Updated documentation: cossacklabs/product-docs#199
Updated engineering demo: cossacklabs/acra-engineering-demo#32

Checklist

@Lagovas Lagovas marked this pull request as ready for review November 17, 2021 14:23
CHANGELOG_DEV.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@vixentael vixentael left a comment

Choose a reason for hiding this comment

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

What will happen if someone will use older config file with auth_keys: configs/auth.keys params present? Will AcraServer show error/warning?

README.md Show resolved Hide resolved
Co-authored-by: vixentael <vixentael@users.noreply.github.com>
@Lagovas
Copy link
Collaborator Author

Lagovas commented Nov 17, 2021

What will happen if someone will use older config file with auth_keys: configs/auth.keys params present? Will AcraServer show error/warning?

will ignore it without any log messages (I have checked it ). acra doesn't validate config for appearing keys that are not used.

…om:cossacklabs/acra into lagovas/T2142-remove-authmanager-webconfig
Our tests check that something handled correctly
according to log output.Aand some of them fork
process with log redirection to file. Before
reading the file called f.Flush to be sure
that all buffered data dumped to FS. But
some logs may be buffered in the process itself that
have written to stdout. And tests don't find
expected log entries in the output.
So now we turn off bufferization
to be sure that we have all available logs.
@Lagovas Lagovas merged commit bb429f3 into master Nov 17, 2021
@Lagovas Lagovas deleted the lagovas/T2142-remove-authmanager-webconfig branch November 17, 2021 18:50
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.

2 participants