Skip to content
This repository has been archived by the owner on Aug 29, 2018. It is now read-only.

Commit

Permalink
oo-app-info: don't assume rhcloud.com app domain
Browse files Browse the repository at this point in the history
Bug 1123944 - oo-app-info does not find gears / users based on app fqdn
https://bugzilla.redhat.com/show_bug.cgi?id=1123944

The search for an app fqdn was poorly implemented, as it assumed the
domain on the fqdn is rhcloud.com. This change generalizes the search to
use the configured cloud domain in the fqdn.
  • Loading branch information
sosiouxme committed Aug 4, 2014
1 parent 4610a75 commit b7b1da5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion broker-util/lib/app_info.rb
Expand Up @@ -87,7 +87,7 @@ def to_s()
end

class AppQuery
FQDN_REGEX = /(http:\/\/)?(\w+)-(\w+)(\.\w+)?\.rhcloud.com\Z/
FQDN_REGEX = /(http:\/\/)?(\w+)-(\w+)(\.\w+)?\.#{Regexp.escape(Rails.application.config.openshift[:domain_suffix])}\Z/

#
# Entrypoint for openshift application query methods
Expand Down

0 comments on commit b7b1da5

Please sign in to comment.