>        @Provides
>        @Singleton
>        @Storage
> -      protected Supplier<URI> 
> provideStorageUrl(RegionIdToURISupplier.Factory factory, @ApiVersion String 
> apiVersion) {
> -         return 
> getLastValueInMap(factory.createForApiTypeAndVersion(ServiceType.OBJECT_STORE,
>  apiVersion));
> +      protected Supplier<URI> 
> provideStorageUrl(RegionIdToURISupplier.Factory factory, 
> +            @ApiVersion String apiVersion,
> +            @Named(LocationConstants.PROPERTY_REGION) String region) {

I'm not sure about this, conceptually there is no reason why it couldn't be 
nullable but trying to set a Properties object value to null will result in a 
NullPointerException. With that in mind and because the region value gets 
injected by guice, I don't know how we can use a null value. What are your 
thoughts?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/31/files#r4859022

Reply via email to