My Sandbox Apps links to not found

JanssenAaron · February 7, 2025, 5:52pm

Hello,

I have noticed at our site, if you have a sandbox passenger application, in the “My Sandbox Apps” if you click the launch button for the app you get get a not found. Batch connect apps, either developer or system launch correctly however.

I can manually navigate to /pun/dev/<appname> and that will work fine. I now have a fixed developer dashboard, after some fumbling with ChatGPT.

The leading theory is that the way the routes are setup, having a ‘.’ (period) is not allowed in usernames in a route. Which normally is not an issue because the username/owner is an optional parameter and seems to only be used when launching sandbox passenger apps.

Maybe this should have been a GitHub issue, but I wasn’t sure if anyone else has noticed this behavior before.

Aaron

jeff.ohrstrom · February 7, 2025, 7:02pm

Hi and welcome!

What’s the appname that’s giving you issues? If I create an app called foo.test I don’t even see it in the list of sandbox apps.

In any case - yea it sounds like a bug somewhere. I’ve never seen it myself, but am seeing odd things playing around with it.

Maybe - Rails will recognize the end of a URL with a dot as the format. I.e., /some/app/url.json is requesting the /some/app/url in JSON format.

You can see here how we turn this off for the file browser so that you can actually access JSON files (i.e., :format => false)

github.com/OSC/ondemand

apps/dashboard/config/routes.rb

cb395604b


      
              resources :launchers do
                post 'submit', on: :member
                post 'save', on: :member
              end
            end
          end
          
          # in production, if the user doesn't have access to the files app directory, we hide the routes
          if Configuration.can_access_files?
            constraints(->(request) { request.params[:fs].to_s.match?(%r{^(?!(edit|api))[^/]+$}) }) do
              get 'files/:fs(/*filepath)' => 'files#fs', :defaults => { :fs => 'fs', :format => 'html' }, :format => false,
                  :as => :files
              put 'files/:fs/*filepath' => 'files#update', :format => false, :defaults => { :fs => 'fs', :format => 'json' }
          
              # TODO: deprecate these routes after updating OodAppkit to use the new routes above
              # backwards compatibility with the "api" routes that OodAppkit provides
              # and are used by File Editor and Job Composer
              get 'files/api/v1/:fs(/*filepath)' => 'files#fs', :defaults => { :fs => 'fs', :format => 'html' },
                  :format => false
              put 'files/api/v1/:fs/*filepath' => 'files#update', :format => false,
                  :defaults => { :fs => 'fs', :format => 'json' }

JanssenAaron · February 7, 2025, 7:23pm

It’s not the appname I was referring too in this case but it likely suffers from the same issue.

The issue is when the launch button links you to this route it appends the username into the owner parameter in the route here.

github.com/OSC/ondemand

apps/dashboard/config/routes.rb

cb395604b


      
            end
          end
          get 'errors/not_found'
          get 'errors/internal_server_error'
          get 'dashboard/index'
          get 'logout' => 'dashboard#logout'
          
          # analytics request appears in the access logs and google analytics
          get 'analytics/:type' => proc { [204, {}, ['']] }, :as => 'analytics'
          
          get 'apps/show/:name(/:type(/:owner))' => 'apps#show', :as => 'app', :defaults => { type: 'sys' }
          get 'apps/icon/:name(/:type(/:owner))' => 'apps#icon', :as => 'app_icon', :defaults => { type: 'sys' }
          get 'apps/index' => 'apps#index'
          
          get 'apps/restart' => 'apps#restart' if Configuration.app_sharing_enabled?
          
          root 'dashboard#index'
          
          # App administration
          scope 'admin/:type', constraints: Authz::AppDeveloperConstraint do
            resources :products, except: :destroy, param: :name, constraints: { type: /dev|usr/ } do

I think you are right about the format getting picked up, so for example when the launch button links to /pun/sys/dashboard/apps/show/ood-example-flask-app/dev/aaron.m.janssen it’s really hitting .../ood-example-flask-app/dev/aaron.m with format = .janssen or maybe a little different because of the double period but you get the gist.

I guess it is also worth noting the the next line down in this file needs the same fix because some app’s icons are getting 404’s because of the same reason.

jeff.ohrstrom · February 7, 2025, 7:36pm

You can change the routes of you developer dashboard. Indeed, you can use your development dashboard to boot other development applications.

I’d suggest maybe making that patch to your development dashboard, then attempting to boot ood-example-flask-app from your development dashboard that has the patch.

It seems like you’re the first developer with a . in their name! At any rate - seems like an easy enough patch for us to make it if works.

Topic		Replies	Views
Is there a place that I can find community batch connect & passenger apps? General Discussion	1	408	June 26, 2023
Creating a Node.js/React.js Passenger App Get Help question	6	498	February 12, 2023
Users missing 'My Interactive Sessions' shortcut in navbar Get Help ondemand2 , question	6	277	April 30, 2023
Not a registered developer Get Help	7	1005	May 26, 2022
Re: attempt to update ondemand; configure Passenger Phusion verbose reporting? Get Help question	3	379	April 4, 2022

My Sandbox Apps links to not found

Related topics