Storage: This doesn't match our existing project ID: hello-rightscale002

[papudesi]

https://github.com/GoogleCloudPlatform/google-cloud-python/blob/4b107613b0fcf139569504e9d44fa4a0793b4fd0/storage/google/cloud/storage/notification.py#L31

[chemelnucfin]

I think the project id specifications have changed. I will try to find out more about what they should be and adjust the regex accordingly.

[chemelnucfin]

Hi @papudesi, dhermes suggested a temporary fix while I try to find the actual project id. I will leave this issue open until I can find the actual requirements.

import re

from google.cloud.storage import notification


notification._PROJECT_PATTERN = r'(?P<project>[a-z]+-[a-z]+\d+)'
notification._TOPIC_REF_PATTERN = notification._TOPIC_REF_FMT.format(
    notification._PROJECT_PATTERN,
    notification._TOPIC_NAME_PATTERN)
notification._TOPIC_REF_RE = re.compile(notification._TOPIC_REF_PATTERN)`

[hrfuller]

Also running into this with an existing project id of the form my-project. The suggested solution in #4467 wouldn't work for it anyway. I will also monkey patch the regex.

[tseaver]

The resource manager API docs say:

The unique, user-assigned ID of the Project. It must be 6 to 30 lowercase letters, digits, or hyphens. It must start with a letter. Trailing hyphens are prohibited.

[dhermes]

@tseaver Nice find! (I assume you are implicitly saying "The current regex doesn't match that.")

[chemelnucfin]

Thanks! I'll fix it.

…

On Wed, Dec 6, 2017, 1:19 PM Danny Hermes ***@***.***> wrote: @tseaver <https://github.com/tseaver> Nice find! (I assume you are implicitly saying "The current regex doesn't match that.") — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#4447 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADzDDMskMUejuLstrMblmlcEi_-pW9YRks5s9wTWgaJpZM4QqHga> .

[tseaver]

The current regex insists on three hyphen-separated tokens, the first two of which must be alpha-only, and the last of which must be numeric-only. A regex which fits the spec I quoted would be:

_PROJECT_PATTERN = r'(?P<project>[a-z0-9-]{5,29}[a-z0-9])'

[tseaver]

FTR, pythex is a cool way to test / debug regexes.

[chemelnucfin]

Closed via #4543