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

Consider adapting SolrWayback for public use #73

Open
8 tasks
anjackson opened this issue Mar 9, 2022 · 1 comment
Open
8 tasks

Consider adapting SolrWayback for public use #73

anjackson opened this issue Mar 9, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@anjackson
Copy link
Contributor

anjackson commented Mar 9, 2022

Beyond internal access, to use SolrWayback as a public service (replacing the faceted search part of ukwa-ui), we need to consider more complex issues:

  • Add localization support: UI internationalization netarchivesuite/solrwayback#23
  • Implement as an accessible, responsive design, e.g. follow Warclight Bootstrap for basic facet design and UI. Consider TailwindCSS for layout with Shoelaces for components, to be consistent with Webrecorder's work.
  • Pages like the Toolbox etc. should also be routed and bookmarkable.
  • Display an error message in the UI when calls the the back-end fail, rather than appearing to still be busy.
  • Security review - make sure all APIs can safely be made public.

As well as some minor changes:

  • Make sure the indexer config default (and in SolrWayback Docker Compose) uses url_norm.
  • Add support for common deployment context headers (X-Forwarded-Proto, X-Forwarded-Host etc.) to SolrWayback, so hardcoding the base URL is no longer necessary.
  • Possible optimisation: Change ArcHTTPResolver to check response for Content-Bytes rather than make two requests per request (where the first probes for Accept-Ranges: bytes (Or at least only check the first time?)

This needs to be weighed up against the difficulties in adapting the off-the-shelf options.

@anjackson anjackson added the enhancement New feature or request label Mar 9, 2022
@anjackson
Copy link
Contributor Author

anjackson commented May 4, 2022

Just noticed that some parts are weird, because facets with counts of 0 are being returned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant