search

HumanDynamics / openPDS-RegistryServer

openpds.media.mit.edu
MIT License
11 stars 15 forks source link

is_pds_up hard-codes the /discovery/ping url. #22

Open bradmontgomery opened 10 years ago

bradmontgomery commented 10 years ago

In apps.base.views.is_pds_up, there's a hard-coded value for a discovery url in the PDS, which unfortunately doesn't have that url defined.

Since these are two separate projects, should this be a setting instead of a hard-coded value?

RogerTangos commented 9 years ago

@brian717 ? The registry server is much more your turf than mine.

brian717 commented 9 years ago

Brad, since you went ahead and re-added that URL on the PDS, the page exists again (its removal was an oversight when the large refactor happened a few weeks ago... sorry about that). I'd say the /discovery/ping url can be part of the openPDS spec, and thus, doesn't need to be configurable - any implementation of openPDS will just be required to have it.