Merge pull request #346 from DigitalWPI/pdfviewer_url_patch_10_14_21

added some additional checks to contraints, also better handling of o…
DigitalWPI · Oct 15, 2021 · c34428d · c34428d
2 parents 77d6550 + 8c00538
commit c34428d
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/config/routes.rb b/config/routes.rb
@@ -51,7 +51,9 @@
   concern :exportable, Blacklight::Routes::Exportable.new
 
   mount PdfjsViewer::Rails::Engine => "/pdfjs", as: 'pdfjs'
-  match '/pdfviewer/:id', to: 'pdfviewer#index', constraints: { id: /.*/ }, as: 'pdfviewer', via: [:get]
+  #match '/pdfviewer/:id', to: 'pdfviewer#index', constraints: { id: /.*/ }, as: 'pdfviewer', via: [:get]
+  get '/pdfviewer/:id', to: 'pdfviewer#index', constraints: { id: /[a-z0-9]{9}/ }
+  get '/pdfviewer/:id/:parent', to: 'pdfviewer#index', constraints: { id: /[a-z0-9]{9}/ }
 
   resources :solr_documents, only: [:show], path: '/catalog', controller: 'catalog' do
     concerns :oai_provider