Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Regenerate Implementation Layer with Interface Scoping Changes #12829

Closed
1 task
jspenc4 opened this issue Jul 6, 2020 · 9 comments · Fixed by Azure/autorest.java#659
Closed
1 task

Regenerate Implementation Layer with Interface Scoping Changes #12829

jspenc4 opened this issue Jul 6, 2020 · 9 comments · Fixed by Azure/autorest.java#659
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. Storage Storage Service (Queues, Blobs, Files)
Milestone
Backlog

Comments

@jspenc4
Copy link

jspenc4 commented Jul 6, 2020

Query/Question
how can i solve
java.security.AccessControlException: access denied ("java.lang.reflect.ReflectPermission" "newProxyInPackage.com.azure.storage.blob.implementation")
at java.security.AccessControlContext.checkPermission(AccessControlContext.java:472)
at java.security.AccessController.checkPermission(AccessController.java:884)
at java.lang.SecurityManager.checkPermission(SecurityManager.java:549)
at java.lang.reflect.Proxy.checkNewProxyPermission(Proxy.java:770)
at java.lang.reflect.Proxy.newProxyInstance(Proxy.java:726)
at com.azure.core.http.rest.RestProxy.create(RestProxy.java:683)
at com.azure.core.http.rest.RestProxy.create(RestProxy.java:665)
at com.azure.storage.blob.implementation.ServicesImpl.(ServicesImpl.java:58)
at com.azure.storage.blob.implementation.AzureBlobStorageImpl.(AzureBlobStorageImpl.java:216)
at com.azure.storage.blob.implementation.AzureBlobStorageBuilder.build(AzureBlobStorageBuilder.java:93)
at com.azure.storage.blob.BlobServiceAsyncClient.(BlobServiceAsyncClient.java:110)
at com.azure.storage.blob.BlobServiceClientBuilder.buildAsyncClient(BlobServiceClientBuilder.java:109)
at com.azure.storage.blob.BlobServiceClientBuilder.buildClient(BlobServiceClientBuilder.java:82)

Why is this not a Bug or a feature Request?
A clear explanation of why is this not a bug or a feature request?

it does seem like a bug to me. however, hoping i misunderstand.

my attempts to edit the java.policy file to allow Reflect Permission are failing.
and in any case, my security team will not let me allow it.

Setup (please complete the following information if applicable):

  • OS: [e.g. iOS] ios
  • IDE : [e.g. IntelliJ] eclipse or intellij
  • Version of the Library used 12.6

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

  • Query Added
  • [x ] Setup information Added
@ghost ghost added needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. customer-reported Issues that are reported by GitHub users external to the Azure organization. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Jul 6, 2020
@alzimmermsft alzimmermsft added the Azure.Core azure-core label Jul 7, 2020
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Jul 7, 2020
@alzimmermsft alzimmermsft self-assigned this Jul 8, 2020
@alzimmermsft
Copy link
Member

Thank you for submitting this issue @jspenc4, I'll take a look into reproducing it and finding possible solutions when dealing with strict SecurityManager configurations.

@alzimmermsft alzimmermsft added the design-discussion An area of design currently under discussion and open to team and community feedback. label Jul 15, 2020
@alzimmermsft alzimmermsft mentioned this issue Jul 15, 2020
Closed
4 tasks
@alzimmermsft alzimmermsft added Client This issue points to a problem in the data-plane of the library. common common module used by all azure SDKs (e.g. client, Mgmt) labels Jul 15, 2020
@joshfree joshfree removed the common common module used by all azure SDKs (e.g. client, Mgmt) label Jul 15, 2020
@joshfree joshfree added this to the Backlog milestone Jul 15, 2020
@alzimmermsft alzimmermsft assigned srnagar and unassigned alzimmermsft Jul 17, 2020
@srnagar srnagar mentioned this issue Jul 20, 2020
Merged
@srnagar srnagar reopened this Jul 20, 2020
@YijunXieMS YijunXieMS self-assigned this Aug 6, 2020
@alzimmermsft
Copy link
Member

@YijunXieMS, the AutoRest V4 code generator was updated by @srnagar in the mentioned PR to generate the Swagger interface as public so it doesn't run into this issue anymore. This is a longer term fix that will get resolved as clients regenerate their Swaggers.

@srnagar srnagar removed their assignment Oct 4, 2020
@alzimmermsft alzimmermsft changed the title [QUERY] reflect permission problem Regenerate Implementation Layer with Interface Scoping Changes Nov 9, 2020
@alzimmermsft
Copy link
Member

Renamed, see #17368 for instructions on resolving this issue.

@alzimmermsft alzimmermsft added Storage Storage Service (Queues, Blobs, Files) and removed Azure.Core azure-core design-discussion An area of design currently under discussion and open to team and community feedback. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Nov 9, 2020
@alzimmermsft alzimmermsft mentioned this issue Nov 9, 2020
Closed
11 tasks
@alzimmermsft
Copy link
Member

@rickle-msft @gapra-msft this should be resolved by updating the code generator used.

@fcofdez
Copy link
Contributor

fcofdez commented Dec 1, 2020

Is there any approximate time when this will be tackled? We would like to avoid giving that permission on our security policy.
Thanks!

@rickle-msft
Copy link
Contributor

@fcofdez I think the earliest would be in our January release, though I can't say for certain whether it will make it in there. I'll update here as I get more information

@amishra-dev amishra-dev added the bug This issue requires a change to an existing behavior in the product in order to be resolved. label Jan 29, 2021
openapi-sdkautomation bot referenced this issue in AzureSDKAutomation/azure-sdk-for-java Feb 9, 2021
CodeGen from PR 12829 in Azure/azure-rest-api-specs
212f919 
[Hub Generated] Review request for Microsoft.Cache to add version stable/2021-03-01 (#12829)

* Adds base for updating Microsoft.Cache from version preview/2020-10-01-preview to version 2021-03-01

* Updates readme

* Updates API version in new specs and examples

* Add support for persistence and fix other typos

* Fix SDK Warning: R2063 OperationIdNounConflictingModelNames

* Fix spellcheck error

* Fix acronym capitalization in descriptions

* Replace local parameters with common-types parameters

* Use naming convention consistent with Azure documentation

* Update word choice, remove deprecated x-ms-code-generation-setting, and fix operationId

* Add x-ms-secret extension and make TLS version a string enum

* Suppress errors about secrets being sent in responses
@alzimmermsft alzimmermsft mentioned this issue Feb 10, 2021
Merged
openapi-sdkautomation bot referenced this issue in AzureSDKAutomation/azure-sdk-for-java Feb 22, 2021
CodeGen from PR 12829 in Azure/azure-rest-api-specs
d3f7ff8 
[Hub Generated] Review request for Microsoft.Cache to add version stable/2021-03-01 (#12829)

* Adds base for updating Microsoft.Cache from version preview/2020-10-01-preview to version 2021-03-01

* Updates readme

* Updates API version in new specs and examples

* Add support for persistence and fix other typos

* Fix SDK Warning: R2063 OperationIdNounConflictingModelNames

* Fix spellcheck error

* Fix acronym capitalization in descriptions

* Replace local parameters with common-types parameters

* Use naming convention consistent with Azure documentation

* Update word choice, remove deprecated x-ms-code-generation-setting, and fix operationId

* Add x-ms-secret extension and make TLS version a string enum

* Suppress errors about secrets being sent in responses
@gapra-msft
Copy link
Member

This will be resolved with this PR #18907

@gapra-msft
Copy link
Member

Hi @fcofdez We recently shipped a beta release of the storage libraries with this fix. Could you please try that out and let me know how it goes?

I believe the version is azure-storage-blob 12.11.0-beta.2 +

@gapra-msft
Copy link
Member

Closing as the issue was resolved with the latest version of GAd storage libraries

@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Milestone
Backlog
Development

Successfully merging a pull request may close this issue.

Add option to make proxy service interface public srnagar/autorest.java
9 participants
@fcofdez @joshfree @amishra-dev @rickle-msft @jspenc4 @YijunXieMS @alzimmermsft @gapra-msft @srnagar