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

Testing hidden folders with RTD #585

Closed
wants to merge 5 commits into from
Closed

Testing hidden folders with RTD #585

wants to merge 5 commits into from

Conversation

choldgraf
Copy link
Member

@choldgraf choldgraf commented Dec 11, 2022

This is just me playing around with staticrypt to learn more about it.

I think I may have found a bug in RTD? I opened this one to ask about it:

@choldgraf
Copy link
Member Author

choldgraf commented Dec 11, 2022

Hmmm for some reason it is correctly finding the HTML, but not actually replacing it with the encrypted version.

https://readthedocs.org/projects/2i2c-team-compass/builds/18882723/

EDIT: Indeed this is a known bug:

@choldgraf choldgraf marked this pull request as draft December 12, 2022 09:46
@humitos
Copy link

humitos commented Jan 13, 2023

📰 We've done some progress on the build.jobs.post_build issue. We hope to review that PR (readthedocs/readthedocs.org#9888), merge and deploy in the following weeks. I will try to come back here and comment when it's deployed 😄 . Stay tuned.

@humitos
Copy link

humitos commented Jan 31, 2023

Hi @choldgraf 👋🏼 . I followed your example to test this out and I got it working at https://test-builds.readthedocs.io/en/staticrypt/ (password is humitos). Let me know if this is useful for you 😄

@choldgraf
Copy link
Member Author

@humitos it worked!

@humitos
Copy link

humitos commented Jan 31, 2023

Great! 💯

I'm curious about what's the use-case behind this idea about using staticrypt. Would you like to expand this? 🙏🏼 In particular, in Read the Docs community where it only supports public repositories. So, putting the HTML content behind a password won't avoid people accessing the content anyways.

On the other hand, on Read the Docs for Business, a similar feature is already built-in in the platform: https://docs.readthedocs.io/en/stable/commercial/sharing.html

If you don't want to share the idea publicly for any reason, please contact me at manuel at readthedocs (.org | .com)

@choldgraf
Copy link
Member Author

In our case, I'm auto-generating dashboards about 2i2c's upstream activity as well as some financial activity with Jupyter Book. The data is not publicly accessible, and some of the dashboards might have sensitive information in them. So the source files don't have anything sensitive, but the built outputs do, which is why I wanted to put them behind a password.

@consideRatio
Copy link
Member

@choldgraf is this a PR to close?

@choldgraf
Copy link
Member Author

@consideRatio yeah let's close it, was a little experiment but I'm not sure if/when we'll get to it so we can always re-open if need be

@choldgraf choldgraf closed this Mar 6, 2024
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.

3 participants