Skip to content

Gradle version change3 #2274

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.

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

Closed
wants to merge 22 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions azure-pipelines/pull-request-validation/pr-msal.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ name: $(date:yyyyMMdd)$(rev:.r)

variables:
- name: BuildParameters.jdkVersion
value: 1.11
value: 1.17
- name: BuildParameters.jdkArchitecture
value: x64
- name: BuildParameters.javaHomeSelection
Expand Down Expand Up @@ -40,9 +40,9 @@ jobs:
persistCredentials: True
- template: azure-pipelines/templates/steps/automation-cert.yml@common
- task: JavaToolInstaller@0
displayName: Use Java 11
displayName: Use Java 17
inputs:
versionSpec: '11'
versionSpec: '17'
jdkArchitectureOption: x64
jdkSourceOption: PreInstalled
- task: CodeQL3000Init@0
Expand All @@ -62,7 +62,7 @@ jobs:
inputs:
tasks: msal:testLocalDebugUnitTest -Plabtest -PlabSecret=$(LabVaultAppCert) -ProbolectricSdkVersion=${{variables.robolectricSdkVersion}} -PmockApiUrl=$(MOCK_API_URL)
javaHomeSelection: $(BuildParameters.javaHomeSelection)
jdkVersion: 1.11
jdkVersion: 1.17
- job: spotbugs
displayName: SpotBugs
cancelTimeoutInMinutes: 1
Expand Down Expand Up @@ -92,6 +92,6 @@ jobs:
inputs:
tasks: clean msal:lintLocalDebug
publishJUnitResults: false
jdkVersion: 1.11
jdkVersion: 1.17

...
2 changes: 1 addition & 1 deletion common
Submodule common updated 74 files
+13 −3 LabApiUtilities/build.gradle
+7 −7 azure-pipelines/pull-request-validation/build-consumers.yml
+5 −4 azure-pipelines/pull-request-validation/common.yml
+2 −1 azure-pipelines/templates/steps/spotbugs.yml
+6 −0 changelog.txt
+1 −1 common-java-root/gradle.properties
+2 −1 common-java-root/gradle/wrapper/gradle-wrapper.properties
+31 −8 common/build.gradle
+3 −4 ...ndroidTest/java/com/microsoft/identity/common/internal/platform/AndroidDevicePoPManagerEncryptionTests.java
+11 −1 common/src/main/java/com/microsoft/identity/common/adal/internal/AuthenticationConstants.java
+4 −5 common/src/main/java/com/microsoft/identity/common/adal/internal/AuthenticationSettings.java
+0 −6 common/src/main/java/com/microsoft/identity/common/adal/internal/cache/ADALOAuth2TokenCache.java
+0 −2 common/src/main/java/com/microsoft/identity/common/crypto/AndroidWrappedKeyLoader.java
+0 −3 common/src/main/java/com/microsoft/identity/common/internal/fido/LegacyFidoActivityResultContract.kt
+25 −5 common/src/main/java/com/microsoft/identity/common/internal/msafederation/google/GoogleSignInProvider.kt
+3 −22 common/src/main/java/com/microsoft/identity/common/internal/msafederation/google/SignInWithGoogleParameters.kt
+3 −11 common/src/main/java/com/microsoft/identity/common/internal/platform/AndroidDevicePopManager.java
+0 −13 common/src/main/java/com/microsoft/identity/common/internal/platform/AndroidKeystoreAsymmetricRsaKey.java
+2 −6 common/src/main/java/com/microsoft/identity/common/internal/platform/AndroidPlatformUtil.java
+0 −5 common/src/main/java/com/microsoft/identity/common/internal/platform/AndroidSecretKeyAccessor.java
+1 −6 common/src/main/java/com/microsoft/identity/common/internal/platform/AsymmetricCipher.java
+0 −5 common/src/main/java/com/microsoft/identity/common/internal/platform/AsymmetricRsaKey.java
+0 −2 common/src/main/java/com/microsoft/identity/common/internal/platform/SymmetricCipher.java
+4 −3 common/src/main/java/com/microsoft/identity/common/internal/providers/oauth2/AuthorizationActivity.java
+0 −316 common/src/main/java/com/microsoft/identity/common/internal/providers/oauth2/AuthorizationActivityFactory.java
+253 −0 common/src/main/java/com/microsoft/identity/common/internal/providers/oauth2/AuthorizationActivityFactory.kt
+54 −0 ...on/src/main/java/com/microsoft/identity/common/internal/providers/oauth2/AuthorizationActivityParameters.kt
+17 −0 common/src/main/java/com/microsoft/identity/common/internal/providers/oauth2/BrokerAuthorizationActivity.java
+0 −100 common/src/main/java/com/microsoft/identity/common/internal/providers/oauth2/CameraPermissionRequest.kt
+163 −0 common/src/main/java/com/microsoft/identity/common/internal/providers/oauth2/CameraPermissionRequestHandler.kt
+116 −71 common/src/main/java/com/microsoft/identity/common/internal/providers/oauth2/WebViewAuthorizationFragment.java
+3 −0 common/src/main/java/com/microsoft/identity/common/internal/result/MsalBrokerResultAdapter.java
+5 −4 common/src/main/java/com/microsoft/identity/common/internal/ui/browser/BrowserAuthorizationStrategy.java
+14 −10 common/src/main/java/com/microsoft/identity/common/internal/ui/webview/AzureActiveDirectoryWebViewClient.java
+15 −12 ...n/src/main/java/com/microsoft/identity/common/internal/ui/webview/EmbeddedWebViewAuthorizationStrategy.java
+5 −17 common/src/main/java/com/microsoft/identity/common/internal/ui/webview/WebViewUtil.java
+0 −7 ...osoft/identity/common/internal/ui/webview/certbasedauth/AbstractSmartcardCertBasedAuthChallengeHandler.java
+2 −17 common/src/main/java/com/microsoft/identity/common/internal/ui/webview/certbasedauth/CertBasedAuthFactory.java
+0 −3 common/src/main/java/com/microsoft/identity/common/internal/ui/webview/certbasedauth/DialogHolder.java
+0 −4 .../microsoft/identity/common/internal/ui/webview/certbasedauth/NfcSmartcardCertBasedAuthChallengeHandler.java
+0 −5 common/src/main/java/com/microsoft/identity/common/internal/ui/webview/certbasedauth/SmartcardPinDialog.java
+0 −4 .../microsoft/identity/common/internal/ui/webview/certbasedauth/UsbSmartcardCertBasedAuthChallengeHandler.java
+0 −3 ...om/microsoft/identity/common/internal/ui/webview/certbasedauth/YubiKitNfcSmartcardCertBasedAuthManager.java
+0 −5 ...om/microsoft/identity/common/internal/ui/webview/certbasedauth/YubiKitUsbSmartcardCertBasedAuthManager.java
+2 −1 ...main/java/com/microsoft/identity/common/internal/ui/webview/challengehandlers/CrossCloudChallengeHandler.kt
+1 −0 ...src/main/java/com/microsoft/identity/common/internal/ui/webview/challengehandlers/SwitchBrowserChallenge.kt
+46 −8 ...ain/java/com/microsoft/identity/common/internal/ui/webview/challengehandlers/SwitchBrowserRequestHandler.kt
+130 −0 ...in/java/com/microsoft/identity/common/internal/ui/webview/switchbrowser/SwitchBrowserProtocolCoordinator.kt
+22 −10 common/src/main/java/com/microsoft/identity/common/internal/ui/webview/switchbrowser/SwitchBrowserUriHelper.kt
+22 −30 common/src/main/java/com/microsoft/identity/common/internal/util/AccountManagerUtil.java
+0 −12 common/src/test/java/com/microsoft/identity/common/adal/internal/net/DefaultConnectionServiceTest.java
+92 −0 common/src/test/java/com/microsoft/identity/common/internal/msafederation/google/GoogleSignInProviderTest.kt
+67 −56 ...src/test/java/com/microsoft/identity/common/internal/providers/oauth2/AuthorizationActivityFactoryTest.java
+46 −2 .../src/test/java/com/microsoft/identity/common/internal/ui/webview/AzureActiveDirectoryWebViewClientTest.java
+121 −0 .../java/com/microsoft/identity/common/internal/ui/webview/challengehandlers/CrossCloudChallengeHandlerTest.kt
+3 −3 ...java/com/microsoft/identity/common/internal/ui/webview/challengehandlers/SwitchBrowserRequestHandlerTest.kt
+25 −12 ...test/java/com/microsoft/identity/common/internal/ui/webview/challengehandlers/SwitchBrowserUriHelperTest.kt
+171 −0 ...ava/com/microsoft/identity/common/internal/ui/webview/switchbrowser/SwitchBrowserProtocolCoordinatorTest.kt
+7 −5 common4j/build.gradle
+2 −1 common4j/gradle/wrapper/gradle-wrapper.properties
+0 −1 common4j/src/main/com/microsoft/identity/common/java/crypto/IDevicePopManager.java
+0 −2 common4j/src/main/com/microsoft/identity/common/java/crypto/SigningAlgorithm.java
+6 −1 common4j/src/main/com/microsoft/identity/common/java/opentelemetry/AttributeName.java
+14 −2 common4j/src/main/com/microsoft/identity/common/java/util/ClientExtraSku.java
+2 −2 common4j/src/test/com/microsoft/identity/common/java/util/ClientExtraSkuTest.java
+2 −1 gradle.properties
+3 −3 gradle/versions.gradle
+1 −1 gradle/wrapper/gradle-wrapper.properties
+2 −2 keyvault/build.gradle
+2 −2 labapi/build.gradle
+9 −4 testutils/build.gradle
+1 −2 testutils/src/main/AndroidManifest.xml
+8 −5 uiautomationutilities/build.gradle
+1 −2 uiautomationutilities/src/main/AndroidManifest.xml
2 changes: 1 addition & 1 deletion gradle.properties
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ org.gradle.daemon=true

# See https://stackoverflow.com/questions/56075455/expiring-daemon-because-jvm-heap-space-is-exhausted
# we must make sure that the total size is <7G, as that's the RAM size of VM on the build pipeline.
org.gradle.jvmargs=-Xmx2048m -XX:MaxPermSize=512m -XX:+HeapDumpOnOutOfMemoryError
org.gradle.jvmargs=-Xmx2048m -XX:MaxMetaspaceSize=512m -XX:+HeapDumpOnOutOfMemoryError

# For OneAuth default abiSelection
abiSelection=x86_64
2 changes: 1 addition & 1 deletion gradle/versions.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ ext {
buildToolsVersion = "28.0.3"

// Plugins
gradleVersion = '7.4.2'
gradleVersion = '8.1.1'
kotlinVersion = '1.7.21'
spotBugsGradlePluginVersion = '4.7.1'
jupiterApiVersion = '5.6.0'
Expand Down
2 changes: 1 addition & 1 deletion gradle/wrapper/gradle-wrapper.properties
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@ distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-7.5-all.zip
distributionUrl=https\://services.gradle.org/distributions/gradle-8.1-all.zip
22 changes: 13 additions & 9 deletions msal/build.gradle
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
plugins {
id 'com.microsoft.identity.buildsystem' version '0.2.3'
id 'com.microsoft.identity.buildsystem' version '0.2.4'
id 'com.android.library'
id 'pmd'
id 'checkstyle'
Expand Down Expand Up @@ -44,7 +44,7 @@ android {
// coreLibraryDesugaring "com.android.tools:desugar_jdk_libs:$rootProject.ext.coreLibraryDesugaringVersion"
// }
// }

namespace "com.microsoft.identity.msal"
compileOptions {
// Flag to enable support for the new language APIs
coreLibraryDesugaringEnabled true
Expand All @@ -71,6 +71,9 @@ android {
project.version = android.defaultConfig.versionName
testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
}
buildFeatures {
buildConfig = true
}

buildTypes {
// testCoverageEnabled flag is set to true to get coverage reports for Android Tests
Expand Down Expand Up @@ -163,7 +166,6 @@ task javadoc(type: Javadoc) {
source = android.sourceSets.main.java.srcDirs
configurations.api.setCanBeResolved(true)
classpath += configurations.api
classpath += project.files(android.getBootClasspath().join(File.pathSeparator))

options.memberLevel = JavadocMemberLevel.PUBLIC
options.addStringOption('Xdoclint:none', '-quiet')
Expand All @@ -172,17 +174,19 @@ task javadoc(type: Javadoc) {
exclude '**/R.java'
destinationDir = reporting.file("$project.buildDir/outputs/jar/javadoc/")
}

afterEvaluate {
tasks.javadoc.classpath += files(project.android.getBootClasspath())
}
// Task to generate javadoc.jar
task javadocJar(type: Jar, dependsOn: javadoc) {
from javadoc.destinationDir
classifier 'javadoc'
archiveClassifier.set('javadoc')
destinationDirectory = reporting.file("$project.buildDir/outputs/jar/")
}

task sourcesJar(type: Jar) {
from android.sourceSets.main.java.srcDirs
classifier 'sources'
archiveClassifier.set('sources')
destinationDirectory = reporting.file("$project.buildDir/outputs/jar/")
}

Expand Down Expand Up @@ -300,8 +304,8 @@ task pmd(type: Pmd) {
source = fileTree('src/main/java')

reports {
xml.enabled = false
html.enabled = true
xml.required = false
html.required = true
}
}

Expand All @@ -322,7 +326,7 @@ afterEvaluate {
publishing {
publications {
msal(MavenPublication) {
from components.distRelease
from components.findByName('distRelease')
groupId 'com.microsoft.identity.client'
artifactId 'msal'
//Edit the 'version' here for VSTS RC build
Expand Down
3 changes: 1 addition & 2 deletions msal/src/main/AndroidManifest.xml
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
package="com.microsoft.identity.msal">
<manifest xmlns:android="http://schemas.android.com/apk/res/android">

<uses-permission android:name="android.permission.INTERNET" />
<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
Expand Down
10 changes: 6 additions & 4 deletions msalautomationapp/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ if (project.hasProperty("localFlights")) {
}

android {

namespace "com.microsoft.identity.client.msal.automationapp"
packagingOptions {
pickFirst 'META-INF/common4j.kotlin_module'
}
Expand All @@ -23,10 +23,12 @@ android {
// Flag to enable support for the new language APIs
coreLibraryDesugaringEnabled true
// Sets Java compatibility to Java 8
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
sourceCompatibility JavaVersion.VERSION_17
targetCompatibility JavaVersion.VERSION_17
}
buildFeatures {
buildConfig = true
}

final String BROKER_HOST = "BrokerHost"
final String BROKER_MICROSOFT_AUTHENTICATOR = "BrokerMicrosoftAuthenticator"
final String BROKER_COMPANY_PORTAL = "BrokerCompanyPortal"
Expand Down
3 changes: 1 addition & 2 deletions msalautomationapp/src/androidTest/AndroidManifest.xml
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
package="com.microsoft.identity.client.msal.automationapp">
<manifest xmlns:android="http://schemas.android.com/apk/res/android">

<uses-permission android:name="android.permission.INTERNET"/>
<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE"/>
Expand Down
5 changes: 3 additions & 2 deletions package-inspector/build.gradle
Original file line number Diff line number Diff line change
@@ -1,12 +1,13 @@
apply plugin: 'com.android.application'

android {
namespace "com.microsoft.inspector"
compileOptions {
// Flag to enable support for the new language APIs
coreLibraryDesugaringEnabled true
// Sets Java compatibility to Java 8
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
sourceCompatibility JavaVersion.VERSION_17
targetCompatibility JavaVersion.VERSION_17
}
compileSdk rootProject.ext.compileSdkVersion
defaultConfig {
Expand Down
5 changes: 3 additions & 2 deletions pop-benchmarker/build.gradle
Original file line number Diff line number Diff line change
@@ -1,12 +1,13 @@
apply plugin: 'com.android.application'

android {
namespace "com.microsoft.identity.common.internal.platform"
compileOptions {
// Flag to enable support for the new language APIs
coreLibraryDesugaringEnabled true
// Sets Java compatibility to Java 8
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
sourceCompatibility JavaVersion.VERSION_17
targetCompatibility JavaVersion.VERSION_17
}
compileSdkVersion rootProject.ext.compileSdkVersion
buildToolsVersion rootProject.ext.buildToolsVersion
Expand Down
3 changes: 1 addition & 2 deletions pop-benchmarker/src/main/AndroidManifest.xml
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
package="com.microsoft.identity.common.internal.platform">
<manifest xmlns:android="http://schemas.android.com/apk/res/android">

<application
android:allowBackup="true"
Expand Down
7 changes: 4 additions & 3 deletions testapps/testapp/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -42,16 +42,16 @@ if(project.hasProperty('otelAriaToken')) {
}

android {

namespace "com.microsoft.identity.client.testapp"
// Bumping up to Java Version 8 due to the addition of YubiKit SDK,
// which uses Version 8. Host apps that use libraries dependent on version 8
// must also integrate version 8.
compileOptions {
// Flag to enable support for the new language APIs
//coreLibraryDesugaringEnabled true
// Sets Java compatibility to Java 8
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
sourceCompatibility JavaVersion.VERSION_17
targetCompatibility JavaVersion.VERSION_17
}


Expand Down Expand Up @@ -89,6 +89,7 @@ android {
}
buildFeatures {
viewBinding true
buildConfig true
}
lintOptions {
abortOnError false
Expand Down
4 changes: 1 addition & 3 deletions testapps/testapp/src/main/AndroidManifest.xml
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,7 @@
-->

<manifest xmlns:android="http://schemas.android.com/apk/res/android"
package="com.microsoft.identity.client.testapp"
xmlns:tools="http://schemas.android.com/tools">

xmlns:tools="http://schemas.android.com/tools">
<!-- Camera permission is required for QR + PIN authorization method w/o broker-->
<uses-feature
android:name="android.hardware.camera"
Expand Down
Loading