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

Doc: @control-whois-storage --store-whois #108

Closed
spirillen opened this issue Aug 28, 2020 · 5 comments · Fixed by #109
Closed

Doc: @control-whois-storage --store-whois #108

spirillen opened this issue Aug 28, 2020 · 5 comments · Fixed by #109
Assignees
Milestone

Comments

@spirillen
Copy link
Contributor

This part of the documentation is rather unclear about what is stored or not.

part: https://github.com/funilrys/PyFunceble/blob/control-whois-storage/docs/usage/from-a-terminal.rst#--store-whois

If you keeps --store-whois as False is the bare minimum like dates(created,changed,expired), dns, who? stored

If --store-whois = true are you then keeping the complete whois print inclusing all the data you could template from .TLD

@funilrys
Copy link
Owner

@spirillen I think I forgot my warning from https://github.com/funilrys/PyFunceble/blob/control-whois-storage/docs/configuration/core.rst#store_whois_record

But yes.

store_whois_record is set by default to False. That means that it will store the expiration date and subject (the system still uses/needs them) but not the full WHOIS record.

If you looked at the new (SQL) database structure, it is the record column of the pyfunceble_whois_record table.

There is indeed room for improvement here 🤔

@spirillen
Copy link
Contributor Author

spirillen commented Aug 28, 2020

Hey @funilrys could you try dump a few sql records of each switches? That might help me in writing the docs to see the full picture by my self.

As you know from our keybase conversation I'm still converting to the new structure, so can't currently put up a test bench

A PS to this one, when I'm looking in the https://github.com/funilrys/PyFunceble/blob/control-whois-storage/docs/components/environment-variables.rst#what-do-we-use-and-why- I'm kind a missing this swicth in the Global settings 😒

@funilrys
Copy link
Owner

Before migration to SQLAlchemy:
2020-08-22_17-17

After migration to SQLAlchemy:
2020-08-22_17-23

Without --store-whois or better said with store_whois_record set to False:
(The missing one is because I removed it from the list to reduce the waiting time while testing other things)
2020-08-28_14-47

With --store-whois or better said with store_whois_record set to True:
2020-08-28_14-49

@funilrys funilrys added this to the 3.3.0 milestone Aug 28, 2020
@funilrys funilrys linked a pull request Aug 28, 2020 that will close this issue
@spirillen
Copy link
Contributor Author

I believe this issue should have been closed with c199b1d

Can you agree with that @funilrys ?

@funilrys
Copy link
Owner

Agree.

funilrys pushed a commit to PyFunceble/rst-docs that referenced this issue Apr 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants