firebase_auth anonymous sign in fails with NPE

I am trying to do anonymous sign in to Firebase:
class _MyHomePageState extends State<MyHomePage> {
  final FirebaseAuth _auth = FirebaseAuth.instance;

  @override
  void initState() {
    super.initState();
    _handleSignIn();
  }

  Future<FirebaseUser> _handleSignIn() async {
    final FirebaseUser user = await _auth.signInAnonymously();
    print("signed in " + user.displayName);
    return user;
  }
}

The exception is:
E/MethodChannel#plugins.flutter.io/firebase_auth(15551): Failed to handle method call
E/MethodChannel#plugins.flutter.io/firebase_auth(15551): java.lang.NullPointerException: Attempt to invoke virtual method 'com.google.android.gms.tasks.Task com.google.firebase.auth.FirebaseAuth.signInAnonymously()' on a null object reference
E/MethodChannel#plugins.flutter.io/firebase_auth(15551): 	at io.flutter.plugins.firebaseauth.FirebaseAuthPlugin.handleSignInAnonymously(FirebaseAuthPlugin.java:292)
E/MethodChannel#plugins.flutter.io/firebase_auth(15551): 	at io.flutter.plugins.firebaseauth.FirebaseAuthPlugin.onMethodCall(FirebaseAuthPlugin.java:86)
E/MethodChannel#plugins.flutter.io/firebase_auth(15551): 	at io.flutter.plugin.common.MethodChannel$IncomingMethodCallHandler.onMessage(MethodChannel.java:200)
E/MethodChannel#plugins.flutter.io/firebase_auth(15551): 	at io.flutter.view.FlutterNativeView.handlePlatformMessage(FlutterNativeView.java:163)
E/MethodChannel#plugins.flutter.io/firebase_auth(15551): 	at android.os.MessageQueue.nativePollOnce(Native Method)
E/MethodChannel#plugins.flutter.io/firebase_auth(15551): 	at android.os.MessageQueue.next(MessageQueue.java:323)
E/MethodChannel#plugins.flutter.io/firebase_auth(15551): 	at android.os.Looper.loop(Looper.java:136)
E/MethodChannel#plugins.flutter.io/firebase_auth(15551): 	at android.app.ActivityThread.main(ActivityThread.java:6776)
E/MethodChannel#plugins.flutter.io/firebase_auth(15551): 	at java.lang.reflect.Method.invoke(Native Method)
E/MethodChannel#plugins.flutter.io/firebase_auth(15551): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1496)
E/MethodChannel#plugins.flutter.io/firebase_auth(15551): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1386)
E/flutter (15551): [ERROR:flutter/shell/common/shell.cc(184)] Dart Error: Unhandled exception:
E/flutter (15551): PlatformException(error, Attempt to invoke virtual method 'com.google.android.gms.tasks.Task com.google.firebase.auth.FirebaseAuth.signInAnonymously()' on a null object reference, null)
E/flutter (15551): #0      StandardMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:551:7)
E/flutter (15551): #1      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:292:18)
E/flutter (15551): <asynchronous suspension>
E/flutter (15551): #2      FirebaseAuth.signInAnonymously (file:///C:/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.8.0+1/lib/src/firebase_auth.dart:85:10)
E/flutter (15551): <asynchronous suspension>
E/flutter (15551): #3      _MyHomePageState._handleSignIn (package:shopping_list_flutter/main.dart:98:43)
E/flutter (15551): <asynchronous suspension>
E/flutter (15551): #4      _MyHomePageState.initState (package:shopping_list_flutter/main.dart:67:5)
E/flutter (15551): #5      StatefulElement._firstBuild (package:flutter/src/widgets/framework.dart:3830:58)
E/flutter (15551): #6      ComponentElement.mount (package:flutter/src/widgets/framework.dart:3696:5)
E/flutter (15551): #7      Element.inflateWidget (package:flutter/src/widgets/framework.dart:2950:14)
E/flutter (15551): #8      Element.updateChild (package:flutter/src/widgets/framework.dart:2753:12)
E/flutter (15551): #9      SingleChildRenderObjectElement.mount (package:flutter/src/widgets/framework.dart:4860:14)
E/flutter (15551): #10     Element.inflateWidget (package:flutter/src/widgets/framework.dart:2950:14)
E/flutter (15551): #11     Element.updateChild (package:flutter/src/widgets/framework.dart:2753:12)
E/flutter (15551): #12     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3732:16)
E/flutter (15551): #13     Element.rebuild (package:flutter/src/widgets/framework.dart:3547:5)
E/flutter (15551): #14     ComponentElement._firstBuild (package:flutter/src/widgets/framework.dart:3701:5)
E/flutter (15551): #15     ComponentElement.mount (package:flutter/src/widgets/framework.dart:3696:5)
E/flutter (15551): #16     Element.inflateWidget (package:flutter/src/widgets/framework.dart:2950:14)
E/flutter (15551): #17     Element.updateChild (package:flutter/src/widgets/framework.dart:2753:12)
E/flutter (15551): #18     SingleChildRenderObjectElement.mount (package:flutter/src/widgets/framework.dart:4860:14)
E/flutter (15551): #19     Element.inflateWidget (package:flutter/src/widgets/framework.dart:2950:14)
E/flutter (15551): #20     Element.updateChild (package:flutter/src/widgets/framework.dart:2753:12)
E/flutter (15551): #21     SingleChildRenderObjectElement.mount (package:flutter/src/widgets/framework.dart:4860:14)
E/flutter (15551): #22     Element.inflateWidget (package:flutter/src/widgets/framework.dart:2950:14)
E/flutter (15551): #23     Element.updateChild (package:flutter/src/widgets/framework.dart:2753:12)
E/flutter (15551): #24     SingleChildRenderObjectElement.mount (package:flutter/src/widgets/framework.dart:4860:14)
E/flutter (15551): #25     Element.inflateWidget (package:flutter/src/widgets/framework.dart:2950:14)
E/flutter (15551): #26     Element.updateChild (package:flutter/src/widgets/framework.dart:2753:12)
E/flutter (15551): #27     SingleChildRenderObjectElement.mount (package:flutter/src/widgets/framework.dart:4860:14)
E/flutter (15551): #28     Element.inflateWidget (package:flutter/src/widgets/framework.dart:2950:14)
E/flutter (15551): #29     Element.updateChild (package:flutter/src/widgets/framework.dart:2753:12)
E/flutter (15551): #30     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3732:16)
E/flutter (15551): #31     Element.rebuild (package:flutter/src/widgets/framework.dart:3547:5)
E/flutter (15551): #32     ComponentElement._firstBuild (package:flutter/src/widgets/framework.dart:3701:5)
E/flutter (15551): #33     StatefulElement._firstBuild (package:flutter/src/widgets/framework.dart:3848:11)
E/flutter (15551): #34     ComponentElement.mount (package:flutter/src/widgets/framework.dart:3696:5)
E/flutter (15551): #35     Element.inflateWidget (package:flutter/src/widgets/framework.dart:2950:14)
E/flutter (15551): #36     Element.updateChild (package:flutter/src/widgets/framework.dart:2753:12)
E/flutter (15551): #37     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3732:16)
E/flutter (15551): #38     Element.rebuild (package:flutter/src/widgets/framework.dart:3547:5)
E/flutter (15551): #39     ComponentElement._firstBuild (package:flutter/src/widgets/framework.dart:3701:5)
E/flutter (15551): #40     ComponentElement.mount (package:flutter/src/widgets/framework.dart:3696:5)
E/flutter (15551): #41     Element.inflateWidget (package:flutter/src/widgets/framework.dart:2950:14)
E/flutter (15551): #42     Element.updateChild (packag

Dependencies are:
  firebase_auth: ^0.8.0+1
  google_sign_in: ^4.0.0

Doctor:
C:\flutter\bin\flutter.bat doctor --verbose
[√] Flutter (Channel stable, v1.0.0, on Microsoft Windows [Version 6.1.7601], locale ru-RU)
    • Flutter version 1.0.0 at C:\flutter
    • Framework revision 5391447fae (9 weeks ago), 2018-11-29 19:41:26 -0800
    • Engine revision 7375a0f414
    • Dart version 2.1.0 (build 2.1.0-dev.9.4 f9ebf21297)

[√] Android toolchain - develop for Android devices (Android SDK 28.0.3)
    • Android SDK at C:/Android_sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = C:/Android_sdk
    • Java binary at: C:\Program Files\Android\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[√] Android Studio (version 3.3)
    • Android Studio at C:\Program Files\Android\android-studio
    • Flutter plugin version 32.0.1
    • Dart plugin version 182.5124
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[!] IntelliJ IDEA Ultimate Edition (version 2018.3)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA 2018.3
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.io/intellij-setup/#installing-the-plugins

[√] Connected device (1 available)
    • SM A310F • 310084d1272f13c7 • android-arm • Android 7.0 (API 24)

! Doctor found issues in 1 category.
Process finished with exit code 0

The log says Flutter plugins not installed but in fact they are installed.

I'm also seeing this crash but with getCurrentUser. It seems like some kind of race condition for the android SDK's FirebaseAuth to get initialized or something. It has worked in the past.

Same NPE with FlutterAuth for me, when calling currentUser. I tried to delay the creation of the instance (by a few seconds) to check if there was any race condition, but it always throws.
Other Firebase services, for instance Firestore and Analytics, work just fine, so I can rule out a misconfiguration. This service seems not to be registered in the FirebaseApp.
Tried on both emulator API28 w/ Play Services and real device.
firebase_auth: ^0.8.1
I'm set up to use Androidx.
build.gradle (project):
dependencies {
classpath 'com.android.tools.build:gradle:3.3.2'
classpath 'com.google.gms:google-services:4.2.0'
}
build.gradle (app):
implementation 'com.google.firebase:firebase-core:16.0.7'
implementation 'com.google.android.gms:play-services-auth:16.0.1'
// no other dependencies
[√] Flutter (Channel beta, v1.2.1, on Microsoft Windows [Versione 10.0.17134.590], locale it-IT)
• Flutter version 1.2.1 at E:\Flutter
• Framework revision 8661d8aecd (4 weeks ago), 2019-02-14 19:19:53 -0800
• Engine revision 3757390fa4
• Dart version 2.1.2 (build 2.1.2-dev.0.0 0a7dcf17eb)
[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at E:\Android\SDK
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• Java binary at: E:\Android\android-studio-35\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
• All Android licenses accepted.
[√] Android Studio (version 3.3)
• Android Studio at E:\Android\android-studio-33
• Flutter plugin version 33.4.1
• Dart plugin version 182.5215
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
[√] Android Studio (version 3.5)
• Android Studio at E:\Android\android-studio-35
• Flutter plugin version 33.3.2
• Dart plugin version 183.5901
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

A lot of people seem to have this issue and the @FlutterIssues team does not say anything. What is up with this? I have not been able to run the app on android for two weeks now.

I'm getting Firebase auth errors. It used to be that you could do Anonymous auth without any of the Google SignIn plugins. Is that no longer the case?

I ran into the same problem a couple of days ago, found this issue and ran 'flutter clean'. This time it worked for my case.

Have this problem for quite some time now, found a solution that makes my project works with firebase auth, hope this helps.

Migrate your flutter project to AndroidX, found out that the template code produced by Flutter SDK are not up to date even if I have the latest version installed.

Go into [project]/android/build.gradle. Change the ext.kotlin_version to 1.3.x

Register your app on Firebase's website, add the config file to your app. (Don't follow the 3rd step written on the Firebase's website. )

Add the classpath to the [project]/android/build.gradle file. (

buildscript {
  dependencies {
    // Add this line
    classpath 'com.google.gms:google-services:4.3.0'
  }
}

Add the apply plugin to the [project]/android/app/build.gradle file.

// ADD THIS AT THE BOTTOM
apply plugin: 'com.google.gms.google-services'

Open your flutter project with Android Studio to Sync the build.gradle file and it firebase auth should be working fine in your Flutter project now.

Here's the build.gradle code of my working project

[project]/android/app/build.gradle
def localProperties = new Properties()
def localPropertiesFile = rootProject.file('local.properties')
if (localPropertiesFile.exists()) {
    localPropertiesFile.withReader('UTF-8') { reader ->
        localProperties.load(reader)
    }
}

def flutterRoot = localProperties.getProperty('flutter.sdk')
if (flutterRoot == null) {
    throw new GradleException("Flutter SDK not found. Define location with flutter.sdk in the local.properties file.")
}

def flutterVersionCode = localProperties.getProperty('flutter.versionCode')
if (flutterVersionCode == null) {
    flutterVersionCode = '1'
}

def flutterVersionName = localProperties.getProperty('flutter.versionName')
if (flutterVersionName == null) {
    flutterVersionName = '1.0'
}

apply plugin: 'com.android.application'
apply plugin: 'kotlin-android'
apply from: "$flutterRoot/packages/flutter_tools/gradle/flutter.gradle"

android {
    compileSdkVersion 28

    sourceSets {
        main.java.srcDirs += 'src/main/kotlin'
    }

    lintOptions {
        disable 'InvalidPackage'
    }

    defaultConfig {
        // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html).
        applicationId "com.example.firebase_retest"
        minSdkVersion 16
        targetSdkVersion 28
        versionCode flutterVersionCode.toInteger()
        versionName flutterVersionName
        testInstrumentationRunner "android.support.test.runner.AndroidJUnitRunner"
    }

    buildTypes {
        release {
            // TODO: Add your own signing config for the release build.
            // Signing with the debug keys for now, so `flutter run --release` works.
            signingConfig signingConfigs.debug
        }
    }
}

flutter {
    source '../..'
}

dependencies {
    implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
    testImplementation 'junit:junit:4.12'
    androidTestImplementation 'androidx.test:runner:1.1.1'
    androidTestImplementation 'androidx.test.espresso:espresso-core:3.1.1'
}

apply plugin: 'com.google.gms.google-services'

[project]/android/build.gradle
buildscript {
    ext.kotlin_version = '1.3.21'
    repositories {
        google()
        jcenter()
    }

    dependencies {
        classpath 'com.android.tools.build:gradle:3.4.2'
        classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
        classpath 'com.google.gms:google-services:4.3.0'
    }
}

allprojects {
    repositories {
        google()
        jcenter()
    }
}

rootProject.buildDir = '../build'
subprojects {
    project.buildDir = "${rootProject.buildDir}/${project.name}"
}
subprojects {
    project.evaluationDependsOn(':app')
}

task clean(type: Delete) {
    delete rootProject.buildDir
}

@autolainen
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/474. Any further collaboration will be done there.


Make Input an owner of transforms

per_sample_transform to be applied in get_item?
per_batch / collate in collate_fn property
device transform property

The above all need to be handled in DataModule

Updated and now propagated to all tasks


How to import it in create-react-app?

Is there a straight way to import the component using 'import from'?

Ok, so I found a way to do it (not sure if itś the best one):
import { Menu, MainButton, ChildButton } from 'react-mfb'
import 'react-mfb/mfb.css'
import '../styles/ionicons.min.css'; //(downloaded from the ionicons.com website and then put it manually into my project)
And then inside the render()  and before return:

nice @shin-monkey


Add aws_apigatewayv2_route_response resource

Reordered apigatewayv2 resources inside the list of supported resources and added the new route_response resource

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[amazon-neptune] Automate EOL retrieval

Based on https://docs.aws.amazon.com/AmazonRDS/latest/UserGuide/MySQL.Concepts.VersionMgmt.html.

Expected diff:
diff --git i/releases/amazon-rds-mysql.json w/releases/amazon-rds-mysql.json
index 81316c7..451182c 100644
--- i/releases/amazon-rds-mysql.json
+++ w/releases/amazon-rds-mysql.json
@@ -1,5 +1,14 @@
 {
-  "releases": {},
+  "releases": {
+    "8.0": {
+      "name": "8.0",
+      "eol": "2026-07-31"
+    },
+    "5.7": {
+      "name": "5.7",
+      "eol": "2024-02-29"
+    }
+  },
   "versions": {
     "8.0.36": {
       "name": "8.0.36",


Add testing to CI

Description
This PR adds a new CI workflow, which now also executes the available tests. Additionally, the pylint parts have been removed in favor of a ruff action, which will do the linting and format checks from now on.


Body Font size and color

The text font size in body section is too small and with the color in the bacgrounf it is less visible assign me this issue with label gssoc2024 and hackoctober 2024

@BhuwanSKumar  can you kindly assign this issue to me


Allow setting Cloudflare proxying on a per-ingress basis

The problem
Cloudflare offers the functionality to proxy all traffic through their servers. Currently, this feature can only be enabled globally, causing it to apply for all created records.
The solution
Using annotations, we can override the globally specified proxying option:
external-dns.alpha.kubernetes.io/cloudflare-proxied: "true"
This closes #370
Attribution
This PR was co-created with @amolenaar

CLA should now be signed by both of us.

/LGTM

@shashidharatd Do you mind having a look again? 🙂

@shashidharatd Any chance you could take a look? 🙏

@eswets sorry again for taking so long to merge this PR, do you mind rebase from the master again to solve the conflicting files?

That should be all!

Is there anything left to be done with this, or only a lgtm is missing?

I'll take a look at the feedback from @unix-way tonight, as this behaviour is not intended. Having said that, I would like to have this merged to prevent having to fix all merge conflicts again.


One sentence per line

Remove trailing white space where it has no meaning.

Than you @kytrinyx .


Elastic Search Index

Hello, how can I modify the elastic search index created by zipkin?
I have three container running for int/qs/live and I want to have three different indices for this in one ES instance.

Use the environment variable ES_INDEX
https://github.com/openzipkin/zipkin/blob/master/zipkin-server/src/main/resources/zipkin-server-shared.yml#L112


browser.get() doesn't work in Safari with Selenium 2.45+

I'm experiencing an issue where browser.get() times out in Safari 100% of the time. I believe the root issue is described well in #2028. This issue was closed, but as far as I can tell, the root problem remains.
Is anyone else experiencing this?
Safari 8.0.4
WebDriver 2.45.0
OS X 10.10.2

Have the same problem here, using
Safari 8.0.5
Webdriver 2.45.0
OSX 10.10.3

+1
Protractor 2.0.0
Safari 8.0.5
Webdriver 2.45.0
OSX 10.10.3

Same here
Protractor: 2.0.0
OS: Windows 7
Webdriver: 2.45.0
Safari: 5.1.7

@eduardomb How confident are you that the problem is window.location.replace?  We would conceivably find some way to detect that we're on a new page (e.g. add some random property to the <html> tag), but I don't like the idea much because
a) It's a hack around an external bug
b) It doesn't even sound like you're sure there isn't some other underlying problem
c) The URL method is very reliable

I'm not 100% positive the problem is window.location.replace. I did narrowed the problem down to this command a while ago but you are right, I'm not sure there isn't some other underlying problem.
I'll try to do some new experiments tomorrow and post the details here.

@sjelin the problem is executeScript does not execute when the page is about:blank in Safari. I don't know why this happens, but it only happens in Safari, so it might be an external bug in SafariDriver.
To illustrate that problem, I've changed the resetUrl to http://google.com and the tests run just fine. (https://github.com/eduardomb/protractor/commit/46ebd896af625f975395d3c27b977c7724f9f563)
You can confirm that it works by running:
cd /tmp/
git clone https://github.com/eduardomb/protractor protractor
cd protractor
git checkout safari_hack
npm install
./bin/webdriver-manager update
./bin/webdriver-manager start &> /dev/null &
npm start &> /dev/null &
export TEST_BROWSER_NAME=safari
npm test

FYI, our tests run on safari on Travis happily: https://travis-ci.org/angular/protractor/jobs/63260687#L309
But that's running with selenium standalone 2.44.0, so it might be a problem with the 2.45 version.

It might be it, @juliemr. Thanks! But how is it possible to use the latest protractor with selenium 2.44.0? Isn't protractor >=2 incompatible with this version?  I've tried to downgrade selenium and now the tests fails with the message Unimplemented function: "then".

@eduardomb it's confusing, but there's two things that can be called 'Selenium'

the selenium-webdriver node module, aka WebDriverJS
The selenium standalone server, which is downloaded via webdriver-manager update
They have versions which roughly follow each other. You can downgrade the selenium standalone server and still work with the latest version of Protractor, but you can't downgrade WebDriverJS.

I can confirm the issue and the fact that for a simple browser.get :

protractor 2.1.0 + selenium-standalone 2.45.0 + safari webdriver extension 2.45 : fails
protractor 2.1.0 + selenium-standalone 2.44.0 + safari webdriver extension 2.45 : works

Thanks for actually doing the test. I wonder if Safari can accept the data url that all the other browsers use now.

Thanks a lot, @juliemr. I've downgraded selenium-standalone to 2.44.0 and tests are running perfectly on Safari. :)

@juliemr I've wrote a simple script using selenium-standalone 2.45 and Safari in order to reproduce the bug and open an issue on Selenium project.
To my surprise, the script is able to successfully execute a driver.get('about:blank') followed by a window.location.replace('http://google.com'). Now, I'm not sure whether this is an external bug or not.
It would be nice if someone more  familiarized with selenium could figure this out. The script runs pretty much the same commands in Protractor.get.
Steps to run the script:
mkdir /tmp/selenium-tst/
cd /tmp/selenium-tst/
npm install selenium-webdriver selenium-standalone
java -jar ./node_modules/selenium-standalone/.selenium/selenium-server/2.45.0-server.jar &> /dev/null &
curl -L http://pastebin.com/raw.php\?i\=G6Wgb41D > script.js
node script.js

Im having the same problem, its working fine with safari extension v2.38.0 but when i upgraded the safari extension to v2.45 it fails.
@eduardomb can you tell me how to dowgrade back to either safari extension v2.38.0 or downgrade the selenium-standalone to 2.44.0

@spatchamatla you can simply download selenium-standalone 2.44.0 here and instead of running the usual webdriver-manager start you should run java -jar selenium-server-standalone-2.44.0.jar.

@eduardomb
Protractor 2.1.0
Safari 8.0.6
Webdriver 2.45.0
OSX 10.10.3
i tried with selenium-server-standalone-2.44.0 and when i run the script it navigates to the first URL but when it try to navigate to second url, it displays only about:blank and it times out with
ScriptTimeoutError: Timed out awaiting response to command "executeScript" after 30001 ms (WARNING: The server did not provide any stacktrace information)
Command duration or timeout: 30.10 seconds
Build info: version: '2.44.0', revision: '76d78cf', time: '2014-10-23 20:02:37'
System info: host: 'mac.local', ip: '<>', os.name: 'Mac OS X', os.arch: 'x86_64', os.version: '10.10.3', java.version: '1.8.0_45'
Driver info: org.openqa.selenium.safari.SafariDriver
Capabilities [{browserName=safari, takesScreenshot=true, javascriptEnabled=true, version=8.0.6, cssSelectorsEnabled=true, platform=MAC, secureSsl=true}]
Session ID: null

its not even navigating to second URL.Im using browser.get() to navigate to the URLs. i tested both url's manually and with chrome as well.
java.lang.NullPointerException
	at org.openqa.selenium.remote.RemoteWebDriver.getCurrentUrl(RemoteWebDriver.java:319)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:497)
	at org.openqa.selenium.support.events.EventFiringWebDriver$2.invoke(EventFiringWebDriver.java:101)
	at com.sun.proxy.$Proxy1.getCurrentUrl(Unknown Source)
	at org.openqa.selenium.support.events.EventFiringWebDriver.getCurrentUrl(EventFiringWebDriver.java:164)
	at org.openqa.selenium.remote.server.handler.GetCurrentUrl.call(GetCurrentUrl.java:30)
	at org.openqa.selenium.remote.server.handler.GetCurrentUrl.call(GetCurrentUrl.java:1)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at org.openqa.selenium.remote.server.DefaultSession$1.run(DefaultSession.java:168)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(Thread.java:745)

Is there any where i can download safari extension v2.38.0?

@spatchamatla it should have worked. Not sure if this will solve your problem, but you can try to run my protractor fork that uses selnium-standalone 2.44.0:
cd /tmp/
git clone https://github.com/eduardomb/protractor
cd protractor
git checkout selenium244
npm install
./bin/webdriver-manager update
./bin/webdriver-manager start
Then use a new window to run your tests:
/tmp/protractor/bin/protractor /path/to/your/test.js

@eduardomb I tried with your branch still the same. what safari Extension are you using? Im using safari Extension v2.45.0
I'm trying to uninstall protractor using npm uninstall  protractor, is there a way to uninstall protractor completely so that i can reinstall it again?

Hello everyone,
Has anyone found a solution for this

i gave some spec in which i can replicate the bug, Unable to navigate to URL in Safari #2346

Same problem here.
Safari 8.0
WebDriver 2.45.0
Selenium 2.44.0
OS X Yosemite

Possibly related?
https://code.google.com/p/selenium/issues/detail?id=7176

I download and install 2.48 version of safari driver.
It seems to work.
http://selenium-release.storage.googleapis.com/index.html?path=2.48/

Starting with Selenium 2.48.0, you must manually install the SafariDriver browser extension.
I also had open up restrictions for localhost.
https://github.com/SeleniumHQ/selenium/wiki/SafariDriver

Looks like we need to give Safari some love. @mgiambalvo assigning you arbitrarily :)

Could you explain what open up restrictions to localhost means? I still can't get this work on Protractor Version 4.0.0 and the latest selenium web driver. I have 2.48.0 installed and enabled in my extensions and selenium keeps looking for......... WebDriverError: Failed to connect to SafariDriver after 10058 ms
[safari 2.48.0 #21] Build info: version: '2.53.1', revision: 'a36b8b1', time: '2016-06-30 17:37:03'

I've had issues getting the Safari extension working properly. It's worth noting that their links to the Safari extension go to their release page, but only the 2.48 release has the extension prebuilt. I found that, while I do have the extension installed and trusted, the browser started by webdriver may or may not have the extension.
It looks like there's a new driver for Safari coming with Sierra. See this issue from SeleniumHQ https://github.com/SeleniumHQ/selenium/issues/2475

Duplicate of #2816. See also selenium/1313 and selenium/1185. Unfortunately, I don't think there's anything we can do about this. Trying it out on my own, I found the Webdriver Safari extension would randomly not work when starting a new Safari instance.
Hopefully, the new Safari driver in Webdriver 3.0 will work better.

Do you know if this bug is still present?
I'm still not able to get safary 12 to work on GlobaLeaks: https://travis-ci.org/globaleaks/GlobaLeaks/jobs/657420618
It fails with: "[18:56:51] E/browser - Could not find Angular on page http://localhost:9000/ : angular never provided resumeBootstrap"


[Windows] fatal "no match for platform in manifest: not found"

Description
I have just installed containerd and nerdctl on windows so I'll run postgres to test but give the error:
time="2024-04-14T18:43:11-03:00" level=fatal msg="no match for platform in manifest: not found"
Steps to reproduce the issue

Installed containerd by following https://github.com/containerd/containerd/blob/main/docs/getting-started.md#installing-containerd-on-windows
Downloaded nerdctl.exe and put it side by side with containerd.exe
Tried to run nerdctl run --name postgres -e POSTGRES_PASSWORD=password -d postgres

Describe the results you received and expected
I expected to start a postgres container running.
What version of nerdctl are you using?
time="2024-04-14T18:54:02-03:00" level=warning msg="unable to determine buildctl version: exec: \"buildctl\": executable file not found in %PATH%"
time="2024-04-14T18:54:03-03:00" level=warning msg="unable to determine runc version: exec: \"runc\": executable file not found in %PATH%"
Client:
 Version:       v1.7.5
 OS/Arch:       windows/amd64
 Git commit:    cffed372371dcbea3dc9a646ce5a913fc1c09513
 buildctl:
  Version:

Server:
 containerd:
  Version:      v1.7.15
  GitCommit:    926c9586fe4a6236699318391cd44976a98e31f1
 runc:
  Version:

Are you using a variant of nerdctl? (e.g., Rancher Desktop)
None
Host information
Client:
 Namespace:     default
 Debug Mode:    false

Server:
 Server Version: v1.7.15
 Storage Driver: windows
 Logging Driver: json-file
 Cgroup Driver:
 Cgroup Version:
 Plugins:
  Log: fluentd journald json-file syslog
  Storage: windows-lcow windows
 Security Options:
 Kernel Version:
 Operating System:
 OSType: windows
 Architecture:
 CPUs: 0
 Total Memory: 0B
 Name: gsevla-desktop
 ID: fa165362-8730-4069-a513-139689eca54e```

postgres on Docker Hub does not seem to have Windows version, so this behavior is expected.
https://hub.docker.com/_/postgres
If you are trying to run Linux containers, you have to run containerd and nerdctl on WSL2


Simple xacro doesn't seem to work

Hi, thanks for the great package!
For some reason, this simple xacro, named test.urdf.xacro, doesn't show anything in jupyter lab:
<?xml version="1.0"? >
<robot xmlns:xacro="http://www.ros.org/wiki/xacro">
  <link name="base_link">
    <visual>
      <geometry>
        <box size="1 1 1" />
      </geometry>
    </visual>
  </link>
</robot>

However, I'm able to view the .xacro in the examples/ repo.
What might be the cause?

Hi @kevin-thankyou-lin!
It seems that the space you have between the ? and the > in the first line is causing a parsing error. When opened with the editor, all the syntax is the same color when it shouldn't be.
Apart from that, there is another issue where this model will not display when first opened. It seems to work fine after the file is edited, but maybe the initial loading of the file has a problem. I have to look into this a bit more.
But hopefully with this you can edit your xacro file :)

Hi @IsabelParedes, seems to work fine after the file is edited, but maybe the initial loading of the file has a problem - indeed! Nonetheless, thanks for help with the xacro fix!


Array size mismatch in numpy kernel

When calling MCEqRun.solve with kernel_config=='numpy'

Traceback (most recent call last):
File "main.py", line ..., in
mceq_run.solve()
File "./MCEq/core.py", line 1123, in solve
self._forward_euler(**kwargs)
File "./MCEq/core.py", line 1307, in _forward_euler
self.solution, self.grid_sol = kernel(*args)
File "./MCEq/kernels.py", line 132, in kern_numpy
mu_egrid, mu_egrid + mu_dEdX * dXaccum,
ValueError: operands could not be broadcast together with shapes (88,) (89,)

Hi Austin,
sorry for silence. This issue had to do with the present version and disappears for different reasons in the release version that I'm working on.
Thanks for pointing this out.


Added support for elasticlib in python

Have you ran this through wpiformat yet? If so then it looks good but I just want to confirm

No, I'll run

Already using it apperantly


Post Request to /convert-token takes more than 10 second

This might be a small trivial issue, but it's taking away good end user experience.
it'll be nice to clear it out. Or at least explain what happens in the background.

Hi @ibrahimyaacob92 . Can you provide me with a deeper explanation? I use the framework myself and I never faced such an issue.

Hi @ibrahimyaacob92 . Can you provide me with a deeper explanation? I use the framework myself and I never faced such an issue.

@wagnerdelima, firstly, thanks on taking on this project ! It's really helpful. I'm trying to work on some other project using this library. I'll get back to you if i manage to replicate this problem.

@wagnerdelima, firstly, thanks on taking on this project ! It's really helpful. I'm trying to work on some other project using this library. I'll get back to you if i manage to replicate this problem.

Cool, thanks.

Cool, thanks.


Low loss and high loss in real life

Let's take an example of Alice stealing our company's customer data, which is worth $1M to us.
Intuitively, the low loss is that she fails to take any data ($0) and the high loss is that she steals all the data ($1M).
Am I choosing the low loss and high loss correctly here? Also, I read that the low loss and high loss should create a 90% confidence interval...how does that apply here?

A $0 loss isn't really a loss, and in any case the lognormal can't capture a 0 low loss accurately (it can't deal well with large high loss/low loss ratios).
Instead when I am eliciting low/high loss values I describe them like this:

In the low loss scenario, imagine everything goes well (other than that a loss occurred)--detection systems worked, we noticed the alarm as soon as is feasible, we stopped further loss, and recovered quickly. A 5th percentile makes sense by that description, because we might become extraordinarily lucky and have an even lower loss than described.

In the high loss scenario, imagine everything goes wrong--we don't detect the loss, and it continues basically as long as it can, up to and including losing all the data. Here a 95th percentile can still make sense because things can always become worse, for example high publicity around the loss, litigation, regulatory fines, etc. beyond what we might expect.

@mdeshon thanks for your reply!

detection systems worked, we noticed the alarm as soon as is feasible, we stopped further loss, and recovered quickly

The way I'm thinking about it is that the loss is correlated with the amount of customer data lost. In the low loss scenario, a "small amount" of customer data is lost, but that feels arbitrary to me.
Should I say 1% of data was stolen for low loss? 5%? 20%?
High loss is easier for me. I just compute the cost of losing the data itself, plus secondary costs like regulatory fines, litigation, loss of employee productivity, hiring forensics firms, etc.  It's a reasonable worst case scenario is how I think about it.

@mdeshon wanted to follow up!


✨[feature Request]: Improving the designing

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I can improve the design or arrangement of this section.Please provide GSSOC Ext 2024 and hactoberfest label to it.
Use Case
I helps me in improving my working with websites
Benefits
No response
Add ScreenShots
No response
Priority
Medium
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

@AyutheCoder give me preview video of your idea.

Somewhat like this and more,it's just a preview.
https://github.com/user-attachments/assets/b70845bf-af41-41ad-8833-b01b7cf6cd09

I could improve the products page too, could you assign it to me .

@AyutheCoder present one looks good no need these changes.


Add hactoberfest tag to the repo

It would be awesome if we can add hactoberfest tag to the repo this will bring in more contributions
especially small enhancements that are easier to perform on such boilerplate repository for people who are trying to start with opensource

Pull-requests on this repo have not been merged in months. There is a fork with recent contributions: https://github.com/pyb4430/full-stack-fastapi-postgresql

Unfortunately adding the Hacktoberfest label to the whole repo brings a lot of spam PRs. I've been adding hacktoberfest-accepted to accepted PRs so people can get the benefits, but adding the label to the repo increases the amount of small and, in many cases, invalid or incomplete PRs. And right now I need to be able to focus on the ones already there and on adding features and fixing things.
In many cases those small and incomplete PRs take up more time reviewing and adjusting than writing the whole thing from scratch, so I've been having to avoid that.
I haven't updated much here on this repo because I wanted to finish some things in SQLModel first, I built SQLModel for this repo, and I want to refactor everything and integrate it, but I haven't had the time, and I haven't been able to finish the things I had pending on SQLModel first. But anyway, hopefully soon.


update dAPI guides

This is the latest on the dAPI branch. I may need to review formatting again as I couldn't host it locally.


Editable row: breakage when form is embedded within a form

When embedding the editable row table/form within another form the edit buttons no longer work.
http://jsfiddle.net/tallsam/cjwt6jhr/
Remove the wrapper form and it works correctly.

Looking at the code, if the element is wrapped by a form (not editable-form), it assumes it's a single editable element and ignores the e-form attribute.
Why do you need to have an editable-form wrapped by a form?

I'm working on a complex reusable form component that is included within another form via a directive. So the component has no knowledge of the other form. In some circumstances the other form may not exist.
-form
  - name
  - size
  - historical-stock
     - year, volume
     - year, volume

If you consider the above, the stock form is a parent form, and the historical stock area would be the table with the editable rows. historical stock is embedded within the stock form using a directive.

I see what you mean about not needing to use editable form - I think I'll switch to using editable popover in this case.
Feel free to close.

OK, glad I could help.  You need to close the issue.

Okay thanks a lot.


Russian translation: improvements of test-fixtures

Q
A

Is bugfix?
yes (doc)

New feature?
no

Breaks BC?
no

Tests pass?
--

Fixed issues
--

Merged. Thank you!


^nth is slow

^nth is very slow. Especially visible on Windows.
Using 7.111.
I can get around it (just stopping using ^nth), still ^nth can be convenient.

makes no sense to me. what are you nth-ing? How long is the list?  Is this a linear list or a nested concepts list.

^nth-ing in a short list of concepts. I'll provide sample code that runs slowly.

topic: ~topicA1 []
topic: ~topicA2 []
topic: ~topicA3 []
topic: ~topicA4 []
topic: ~topicA5 []
topic: ~topicB1 []
topic: ~topicB2 []
topic: ~topicB3 []
topic: ~topicB4 []
topic: ~topicB5 []
topic: ~topicC1 []
topic: ~topicC2 []
topic: ~topicC3 []
topic: ~topicC4 []
topic: ~topicC5 []
topic: ~topicD1 []
topic: ~topicD2 []
topic: ~topicD3 []
topic: ~topicD4 []
topic: ~topicD5 []
topic: ~topicE1 []
topic: ~topicE2 []
topic: ~topicE3 []
topic: ~topicE4 []
topic: ~topicE5 []

concept: ~testc [
~topicA1
~topicA2
~topicA3
~topicA4
~topicA5
~topicB1
~topicB2
~topicB3
~topicB4
~topicB5
~topicC1
~topicC2
~topicC3
~topicC4
~topicC5
~topicD1
~topicD2
~topicD3
~topicD4
~topicD5
~topicE1
~topicE2
~topicE3
~topicE4
~topicE5
]

topic: ~debut repeat []

t: ^keep()
  $_size = ^length(~testc)
  $_index = 0
  loop() {
    ^nth(~testc $_index)
    $_index += 1
    if ($_index > $_size) {
      End(loop)
    }
  }

This takes 6 seconds to run on my laptop.

takes me no real time at all on my i5 laptop.

Tested again to be sure, 7.111, still have the problem, let's say 5 seconds.
My colleague had the same problem on his laptop. We had to remove all the calls to ^nth in our bot.


add CI

This adds CI checking for future PRs and fixes lints

Just cleanups for the CI/CD to pass


Need to clear tool tip when disabling a button the mouse is over

When a button is disabled that the mouse is over the tooltip is still visible.

This has been fixed.


Add ContentHash struct, replace TransactionalContentMD5 and TransactionContentCrc64 with TransactionalContentHash

Pull Request Checklist
Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:
See the detailed list in the contributing guide.

[x] C++ Guidelines
[x] Doxygen docs
[x] Unit tests
[x] No unwanted commits/changes
[x] Descriptive title/description

[x] PR is single purpose
[x] Related issue listed

[x] Comments in source
[x] No typos
[x] Update changelog
[x] Not work-in-progress
[x] External references or docs updated
[x] Self review of PR done
[x] Any breaking changes?

Split https://github.com/Azure/azure-sdk-for-cpp/pull/1111 into two separate PRs, one for Blob service and one for DataLake/File service. So that other changes to blob protocol layer is unblocked

/azp run cpp - storage

/azp run cpp - storage

/azp run cpp - storage

/azp run cpp - storage

/azp run cpp - storage


difference with queryParameters in NavigationParams and toUrl()

2.0.0-dev.48.0 Dart SDK
Angular 5.0.0-alpha+10
Chrome

My route definition
final searchPath = RoutePath(path: "search");
final _searchRouteDefinition = RouteDefinition.defer(
  routePath: searchPath,
  loader: () async {
    await search_view.loadLibrary();
    return search_view.searchNgFactory;
  },
);

I need to pass queryParameters when navigating to this route, at first I was using toUrl function
final params = { "foo": "bar" };
final url = searchPath.toUrl(queryParameters: params); // url == "/search?foo=bar"
router.navigate(url);

But the router is not navigating to my view and by using NavigationParams
final url = searchPath.toUrl(); // url == "/search"
router.navigate(url, new NavigationParams(queryParameters: params));

It is navigating well to localhost:8080/#search?foo=bar.
Is it the normal behaviour of the navigate function ? I am also using <base href="/">

/cc @leonsenft

Yes, it's intended that query parameters and fragment identifiers are passed via the optional NavigationParams parameter.
Due to popular demand we're adding a navigateByUrl method which will extract the query parameters and fragment identifier from the URL string. It will likely be available in the next alpha release.
That said, in your example you should continue using NavigationParams instead of toUrl(). Using toUrl() means you'd be doing double the work to serialize your query, only to have the router immediately de-serialize it.


Feat(eos_cli_config_gen): Add support for logging format rfc5424

Change Summary
This PR adds support for logging format rfc5424
Related Issue(s)
Fixes #3365
Component(s) name
arista.avd.eos_cli_config_gen
Proposed changes
This adds support for logging format rfc5424.
It adds the templates, documentation and schemas for this.
How to test
I'm not sure how to run tests. I'm using vscode and the dev-container included in the repo.
I'm familiar with Makefiles from *BSD, but not while developing.
See separate comment below.
Checklist
User Checklist
Repository Checklist

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation and documentation have been updated accordingly.
[x] I have updated molecule CI testing accordingly. (check the box if not applicable)

I found the link to molecule in the PR checkbox comment. Looking through that I couldn't find a test for logging format sequence-numbers so I figure that this change is as simple since I followed sequence-numbers and it's changes.
If there still is a need for a test I'll try to make one with molecule.

Moving to draft until we have tests in here. You can mark is as "ready for review" when it is ready.
For Molecule, you should add the new key to ansible_collections/arista/avd/molecule/eos_cli_config_gen/inventory/host_vars/logging.yml and run Molecule locally:
cd ansible_collections/arista/avd
molecule converge -s eos_cli_config_gen -- --limit logging

Moving to draft until we have tests in here. You can mark is as "ready for review" when it is ready.
For Molecule, you should add the new key to ansible_collections/arista/avd/molecule/eos_cli_config_gen/inventory/host_vars/logging.yml and run Molecule locally:
cd ansible_collections/arista/avd
molecule converge -s eos_cli_config_gen -- --limit logging

PLAY [Converge] ****************************************************************

TASK [arista.avd.eos_cli_config_gen : Verify Requirements] *********************
AVD version 4.5.0-dev1
Use -v for details.
ok: [logging -> localhost]

TASK [arista.avd.eos_cli_config_gen : Create required output directories if not present] ***
ok: [logging -> localhost] => (item=/workspaces/ansible-avd/ansible_collections/arista/avd/molecule/eos_cli_config_gen/intended/structured_configs)
ok: [logging -> localhost] => (item=/workspaces/ansible-avd/ansible_collections/arista/avd/molecule/eos_cli_config_gen/documentation)
ok: [logging -> localhost] => (item=/workspaces/ansible-avd/ansible_collections/arista/avd/molecule/eos_cli_config_gen/intended/configs)
ok: [logging -> localhost] => (item=/workspaces/ansible-avd/ansible_collections/arista/avd/molecule/eos_cli_config_gen/documentation/devices)

TASK [arista.avd.eos_cli_config_gen : Include device intended structure configuration variables] ***
skipping: [logging]

TASK [arista.avd.eos_cli_config_gen : Generate eos intended configuration] *****
changed: [logging -> localhost]

TASK [arista.avd.eos_cli_config_gen : Generate device documentation] ***********
changed: [logging -> localhost]

PLAY RECAP *********************************************************************
logging                    : ok=4    changed=2    unreachable=0    failed=0    skipped=1    rescued=0    ignored=0

Should I commit the changes? There are a lot of deletes because of --limit logging...

It should not delete anything. It sometimes happens if it is the first time you run it. So try to run without limit and then commit the updates.

Thanks, ran the command again without --limit logging and it recreated the cfg-files. Ran it again with --limit to be sure and committed and pushed the changes.

We don't care about your commits in your branch. We will squash them into a single commit using the PR title, once we merge it to the devel branch.

Well, tests passed and the workflow passed earlier once I corrected capitalization in the schema.
I think I'm ready for review!
Thank you so much for your help! I couldn't have managed the last mile without your help and guidance!


Compared to STN

How is box convolutions different than Spatial Transformer networks? Isn't that also a way of generating boxes of interest?

Honestly, I can't see how STNs are "a way of generating boxes of interest". It's just a plane transformation.
To my view, that idea isn't even close. For one, spatial transformer isn't a convolution.
The closest idea that you could probably somehow compare box convolutions to is maybe ROI warping...but our approach is still distant.

Thanks for the clarification, Egor.


Incorporate in Space_editor

Hey @nicopap
Are you interested in incorporating this work into SpaceEditor?
You have done quite a lot and we could help you maintain this up to date and develop there

Based on our testing from more active development around that plugin, I don't think that it really makes sense to include it. Loader is very much slow (takes minutes for basic stuff), triangulator supports only basic shapes and so on. You will have to resolbe all the present issues and mayby even contribute a lot of thigs to fbxcel ceate used to parse FBX. One pf possibilities to make this loader fast, is to use Autodesk's official SDK, for example Remedy uses it in open source project for reading it as geometry from USD without any issues.
As a side note, Aurodesk is about to deprecate FBX in favor of Pixar's USD once they will merge specifc set of features.

That makes sense, thanks


fix: can not extract argument valid rule if multiple rules existed

Requirements

Fill out the template below.
The pull request must update the test suite.
After you create the pull request, all status checks must be pass.

Type

[x] Bug fix
[ ] Feature change
[ ] Performance
[ ] Refactor
[ ] Documentation
[ ] CI/CD
[ ] Test
[ ] Build
[ ] Style
[ ] Chore

Issue (only for type bug fix/feature change)
#34
Description of the Change
See in issue.
Possible Drawbacks
None.

:tada: This PR is included in version 1.4.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v1.4.5

Your semantic-release bot :package::rocket:


Client-Side Macros (Scripts & Commands)

So... this was briefly discussed in the official Talecraft chat, and might not even be implemented because of some serious security and griefing concerns.
The idea is to allow the user to execute commands and scripts, by binding them to some trigger. For the start, a simple command should be enough, with a GUI being added on top later.
Note that macros need to be stored on the client-side (!) in some kind of database.
Proto-Command:
  /tcc_setmacro <macro_name:string> <:TRIGGER> <:EXEC>

  :TRIGGER
    key <:key>
    cmd <:string>
    item <:item>
    action <:action>

  :EXEC
    script <script_name>
    cmd <command>
    key <key_name>
    raw {JSON:NBT}

Command Storage Structure (JSON/NBT):
{
  name: "Hello, Explosion!",
  trigger: {
    cmd: "hi_and_boom"
  }
  commands:[
    {delay:0, key:"SPACE"},
    {delay:0, cmd:"say Hi!"},
    {delay:40, cmd:"/explosion ~ ~ ~ 100"}
  ]
}

Small idea from here: A alias command.
Syntax(es):
/alias new NAME COMMAND
/alias delete NAME
/alias list

After creating a new alias, they can be called by writing /. directly followed by the name of the alias.
All aliased are stored on the client side. The server does not know about them.

Because of this, they are 'global' for the client; not saved per world.

There could be a GUI for managing the aliases.


pkgsStatic.borgbackup fails

I tried to build a static version of borgbackup, but it failed:
$ nix build pkgsStatic.borgbackup -f .
…
builder for '/nix/store/8xji6vq74qz05bansmk362vfqi005dgg-keyutils-1.6-x86_64-unknown-linux-musl.drv' failed with exit code 1; last 10 log lines:
  grep: /usr/lib/rpm: No such file or directory
  x86_64-unknown-linux-musl-gcc -I. -g -Wall -Werror -o keyctl.o -c keyctl.c
  x86_64-unknown-linux-musl-gcc -I. -DPKGBUILD="\"1970-01-01\"" -DPKGVERSION="\"keyutils-1.6\"" -DAPIVERSION="\"libkeyutils-1.8\"" -g -Wall -Werror -fPIC -o keyutils.os -c keyutils.c
  x86_64-unknown-linux-musl-gcc -g -Wall -Werror -fPIC  -shared -Wl,-soname,libkeyutils.so.1 -Wl,--version-script,version.lds -o libkeyutils.so.1.8 keyutils.os
  /nix/store/b2zlnsp56ccrq2y0ybgwlm3m2ils7xhj-x86_64-unknown-linux-musl-binutils-2.31.1/bin/x86_64-unknown-linux-musl-ld: /nix/store/zyrpxbdzmqpi2yvxypxkwsp5d2z3g05j-x86_64-unknown-linux-musl-stage-final-gcc-debug-7.4.0/lib/gcc/x86_64-unknown-linux-musl/7.4.0/crtbeginT.o: relocation R_X86_64_32 against hidden symbol `__TMC_END__' can not be used when making a shared object
  /nix/store/b2zlnsp56ccrq2y0ybgwlm3m2ils7xhj-x86_64-unknown-linux-musl-binutils-2.31.1/bin/x86_64-unknown-linux-musl-ld: /nix/store/zyrpxbdzmqpi2yvxypxkwsp5d2z3g05j-x86_64-unknown-linux-musl-stage-final-gcc-debug-7.4.0/lib/gcc/x86_64-unknown-linux-musl/7.4.0/crtend.o: relocation R_X86_64_32 against `.ctors' can not be used when making a shared object; recompile with -fPIC
  /nix/store/b2zlnsp56ccrq2y0ybgwlm3m2ils7xhj-x86_64-unknown-linux-musl-binutils-2.31.1/bin/x86_64-unknown-linux-musl-ld: final link failed: nonrepresentable section on output
  collect2: error: ld returned 1 exit status
  make: *** [Makefile:138: libkeyutils.so.1.8] Error 1
  builder for '/nix/store/8xji6vq74qz05bansmk362vfqi005dgg-keyutils-1.6-x86_64-unknown-linux-musl.drv' failed with exit code 2
cannot build derivation '/nix/store/1amll33h5m5c1w5lvd08iz3rjilzkfq1-libkrb5-1.17-x86_64-unknown-linux-musl.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/p0lqychn6d30nh4kfd0i77xi984m6nqx-openssh-7.9p1-x86_64-unknown-linux-musl.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/28fcvpfy6db21vcw3sckn2pw7rb8b75w-borgbackup-1.1.10-x86_64-unknown-linux-musl.drv': 1 dependencies couldn't be built

This is with nixpkgs revision 6cc7e33ac8ac7b255bd621e8fbfb29c1fd49966e
It is not super important to me, so if reports about a random package not building statically are not helpful at this point, feel free to just close this.

Looks like keyutils is not able to build the static library.
krb5 has a conditional to disable keyutils on bionic and llvm
https://github.com/NixOS/nixpkgs/blob/5ad0677aec2455e664992d93d063d5cb34a32e2f/pkgs/development/libraries/kerberos/krb5.nix#L44
We could either add a musl condition there, or apply cross-compilation tweaks to keyutils. (Gentoo seem to has this.)

I just needed keyutils to get something else built statically. I managed to get it building with the following patch:
diff --git a/pkgs/os-specific/linux/keyutils/default.nix b/pkgs/os-specific/linux/keyutils/default.nix
index 553b0b87f41..1139fe5b5f4 100644
--- a/pkgs/os-specific/linux/keyutils/default.nix
+++ b/pkgs/os-specific/linux/keyutils/default.nix
@@ -23,10 +23,14 @@ stdenv.mkDerivation rec {
     ./conf-symlink.patch
   ];

+  postPatch = ''
+    sed -ie 's/\-shared/\-static/g' Makefile
+  '';
+
   BUILDDATE = "1970-01-01";
   outputs = [ "out" "lib" "dev" ];

-  installFlags = [
+  makeFlags = [
     "ETCDIR=$(out)/etc"
     "BINDIR=$(out)/bin"
     "SBINDIR=$(out)/sbin"
@@ -35,6 +39,8 @@ stdenv.mkDerivation rec {
     "INCLUDEDIR=$(dev)/include"
     "LIBDIR=$(lib)/lib"
     "USRLIBDIR=$(lib)/lib"
+    "NO_ARLIB=0"
+    "NO_SOLIB=1"
   ];

   meta = with stdenv.lib; {

I didn't do further research on this, however e.g. openssh seems to break on pkgsStatic as well.

@Ma27 You should open a PR with that patch (changes conditional on stdenv.targetPlatform.isStatic)!


soundcard access error when re-running a script

When stopping and re-running a script in Spyder, I get a "device unavailable" error. On the first run, there is no sound card access issue.

We have the same issue. A workaround is to close the python interpreter that the script was running in (while keeping Spyder open).


Search Indexing page takes long time to load

Affected module
UI
Describe the bug

To Reproduce
Screenshots or steps to reproduce
Expected behavior
A clear and concise description of what you expected to happen.
Version:

OS: [e.g. iOS]
Python version:
OpenMetadata version: [e.g. 0.8]
OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[docker]==XYZ]

Additional context
Add any other context about the problem here.

@Ashish8689 check this on sandbox. It takes forever to load that. We need to identify where the issue is

@harshach  Have verified it on Sandbox and the reindex API https://sandbox.open-metadata.org/api/v1/search/reindex/latest is taking time to complete that's why we are seeing loading on card!
https://user-images.githubusercontent.com/66266464/234762218-68b455bd-e241-4744-b456-54794dbdb66f.mov

new refresh looks like fixed it. Not sure whats the root cause for it. we will re-open if it happens again


Introductory YouTube video removed?

At  - https://www.youtube.com/watch?v=u0_vkWYYXRc - I get "Video has been removed by the user".
Was just about to sit down and catch up with it!
(Linked in readme.md)

Thanks for pointing that out! I think it was moved and is now at https://www.youtube.com/watch?v=Lws0zOaseIM

Thanks for the find!
Being addressed here https://github.com/aspnet/SignalR/pull/2637

Done here: a2942ce8851c3e504ad51502bec41886df5d5733
Thanks again for point this out @kierenj!


Premake5 Warnings - Deprecated Functions

Premake5 warns of two functions which will be deprecated in future versions:
dependencies\recastnavigation_p5.lua:70
premake5.lua:266

P:\Programming\Nuake>premake5 vs2022
** Warning: the flags value StaticRuntime has been deprecated and will be removed.
   Use `staticruntime "On"` instead
   @...ramming\Nuake\Nuake\dependencies\recastnavigation_p5.lua(70)

** Warning: the flags value WinMain has been deprecated and will be removed.
   Use `entrypoint "WinMainCRTStartup"` instead
   @P:\Programming\Nuake\premake5.lua(266)

Priority: Low
Fix Tasks:
Update dependencies\recastnavigation_p5.lua:70 to use staticruntime "On"
Update premake5.lua:266 to use entrypoint "WinMainCRTStartup"
Expected Result:
No Warnings

Thank you! :)


Has 2.7.11 been validated against .NET 6?

I have an application running IronPython 2.7.11 with .NET 5 without problems. But it isn't running successfully in .NET 6. I don't have a small test program to demo the issue--I was just wondering if 2.7.11 has been validated to work with .NET 6 yet?

@JohnTenney I have a .NET application that runs <PackageReference Include="IronPython.StdLib" Version="3.4.0-alpha1" />
Not really what you asked about, but at least  that version works with .NET6.

I am aware of some small incompatibilities with .NET 6 but (hopefully) none that should prevent an app from working. If you have more info it would be useful as I'm slowly preparing the next release...

Looks like the problem was that ctypes.windll.user32.GetSystemMetrics now returns null.

I think this is caused by one of the incompatibilities I mentioned. I should have a fix which I can port over from the ipy3 repo.


fefwefewfewfwefew

[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9615c579-305d-2482-7eaf-2d120c75e8d5
Version Independent ID: 064f4a10-bd74-f554-9550-781dd3e838cb
Content: Concepts - Networking in Azure Kubernetes Services (AKS) - Azure Kubernetes Service
Content Source: articles/aks/concepts-network.md
Service: container-service
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

Hello @sudarshan-lightmatter, can you please elaborate more on the issue faced?

Please ignore. It was added by mistake.


importing ProfileReport seems to interfere with pandas plotting

Describe the bug
Running
from pandas_profiling import ProfileReport
import pandas as pd

seems to interfere with pandas plotting
To Reproduce
Run this as a cell in jupyter lab (may be same as jupyter notebook)
import pandas as pd
from pandas_profiling import ProfileReport

df = pd.DataFrame(data={"col1": [1, 2]})
df.plot()

Now the same but comment out pandas_profiling

Version information:
pandas-profiling          3.1.0

Good catch. Might have to do with the setting of the backend in 3.1.0.
Two ways of checking this: check if the same happens with 3.0.0 or set the backend of matplotlib after importing PP.

I can confirm the same does not happen on 3.0.0. An easy place to verify is a binder link e.g.
https://mybinder.org/v2/gh/scikit-learn/scikit-learn/1.0.X?urlpath=lab/tree/notebooks/auto_examples/release_highlights/plot_release_highlights_1_0_0.ipynb
!conda install -c conda-forge pandas-profiling --y
import pandas as pd
from pandas_profiling import ProfileReport

df = pd.DataFrame(data={"col1": [1, 2]})
df.plot()


DRAFT - ✨ Allow users to exclude given models/rows

This is a:

[ ] bug fix PR with no breaking changes
[x] new functionality

Link to Issue
Closes #135
Description & motivation
Open for comments, on the idea as well as the implementation.
This is really just a simple POC for now but the extra code in the fct could be moved to a macro.
Advantages:

flexibility for the users to exclude whatever row
only one seed to maintain and not 1 per fct model

Integration Test Screenshot

Checklist

[ ] I have verified that these changes work locally on the following warehouses (Note: it's okay if you do not have access to all warehouses, this helps us understand what has been covered)

[ ] BigQuery
[ ] Postgres
[ ] Redshift
[ ] Snowflake

[ ] I have updated the README.md (if applicable)
[ ] I have added tests & descriptions to my models (and macros if applicable)
[ ] I have added an entry to CHANGELOG.md

Just thinking about it, we could also change the logic

from {{ row_filter["column_name"] }} <> '{{ row_filter["id_to_exclude"] }}'
to {{ row_filter["column_name"] }} not like '{{ row_filter["id_to_exclude"] }}' , allowing users to mention what pattern should be excluded.

This could make sense for the staging models called stg_xxx__unioned which union multiple sources together and currently are raising warnings in fct_multiple_sources_joined (see issue #20 )

Consideration discussed previously with the team.

make sure that there is no impact on performance

I will test it on IA

write docs that make it easy for people to understand what to do to use this feature (part of the customization part of the README)

I still need to update the README with instructions but I'd be keen for some feedback already


Make other wkb and wkt variants generic

This is a follow up to https://github.com/georust/geozero/pull/171 to make other WKB and WKT variants generic over any AsRef<[u8]> input
With this change, I think WktString and WktStr are redundant, but I didn't want to make a breaking change. I think this should be non-breaking because String and &str can AsRef to [u8] right?

Is the failing CI still a change in this PR? It seems unrelated

Is the failing CI still a change in this PR? It seems unrelated

I think this will be fixed here https://github.com/georust/geozero/pull/189

I haven't worked with deprecated objects in rust before and I don't know the right way to get clippy happy with deprecated but not yet removed objects. Thoughts?

I haven't worked with deprecated objects in rust before and I don't know the right way to get clippy happy with deprecated but not yet removed objects. Thoughts?

I would slap some #[allow(deprecated)] on our own internal usage.

Rather than wait for diesel to make a release, I say we merge this and accept that it's a small breaking change to our diesel integration (we're already looking at a semver breaking release as it is).

I say we merge this

Sorry to be sloppy with my language, I mean to say that we should revert the default trait implementation that I earlier suggested, since it's breaking the current release of diesel.
And then merge this PR.

Do you want to keep the deprecated structs instead of removing them in this release?

Do you want to keep the deprecated structs instead of removing them in this release?

Even though this is already going to be a breaking release, I'd prefer to keep them. As a user, I know I like having deprecations that tell me exactly what needs to change.

Awesome, any other comments on this PR?

When do we plan to release the new version on crates.io?


fixed typo in private.go file

Fixed a typo in the error message return of the runGITCommand() function in the private.go file

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 78.20%. Comparing base (b1595c5) to head (d5a9555).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #3   +/-   ##
=======================================
  Coverage   78.20%   78.20%
=======================================
  Files           5        5
  Lines         390      390
=======================================
  Hits          305      305
  Misses         60       60
  Partials       25       25

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Test hosted site

Once we have the newest site up and running attempt to break that.

The hosted site as it is now is working. Once we get the newest version up I will test again.
[t:60]


Release/azure communication phonenumbers/beta dnr [Beta release for DNR]

Packages impacted by this PR
Issues associated with this PR
Describe the problem that is addressed by this PR
Added DNR changes to help acquire numbers from DNR countries
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Yes, Tests are added
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


Use a mixin for setting bs css variables

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the contributing guidelines

Proposal
In my opinion it would be nice to use some kind of a mixin to set bs css variables. For example we could use this mixin to set css variables. In this way users don't require to start the var name with the $prefix variable when they are customizing bootstrap to their own needs.
@mixin set-var($name, $value) {
    --#{$prefix}#{$name}: #{$value};
}

.card-primary {
$background: blue;
$border: if($color == $color-contrast-light, shade-color($background, 10%), tint-color($background, 10%));
$color: color-contrast($background);

    @include set-var(card-border-color, $border);
    @include set-var(card-cap-bg, $background);
    @include set-var(card-cap-color, $color);
    @include set-var(card-color, $color);
    @include set-var(card-bg, tint-color($background, 5%));
}

Motivation and context
I see a lot of repetitive variables using the $prefix variable directly in sass. In my opinion this could be better, so that users who like to customize bootstrap don't have to do much

Could I participate in this issue?

Thanks for this feature request @stijn7621.
I know that we talked about something like this during the implementation of the color mode with @mdo.
My personal opinion was that I don't see why it would be more helpful when you customize Bootstrap.
Copying and pasting @include set-var() instead of --#{$prefix} is as repetitive.
It would bring up something very specific to Bootstrap instead of using the regular declaration of custom properties. I'm not so sure in terms of readability and usage that it would be better TBH.
Moreover, in the documentation (e.g. https://twbs-bootstrap.netlify.app/docs/5.3/components/accordion/#variables), we would have a list of @include set-var() which doesn't look like custom properties.
But this is only my own opinion, let's see what others think about it :)
@Blueclaus13, before launching any dev, let's wait if we agree on this new feature

Thanks @julien-deramond for your answer.
I agree in terms that with set-var it's also repetitive. However if you alter it a bit, to let it also accept key-value pair mappings, this feature could be more user-friendly. Besides, I think using a mixin instead makes the code more understandable and it gives the developer an extra fail-safe in case they forgot/misspelled the #{$prefix} or bs to set/overwrite the css variable. Below is an alternated suggestion, which also accepts keyvaluepair mappings:
@mixin set-vars($values) {
@each $key, $value in $values {
@include set-var($key, $value);
}
}
.card-primary {
@include set-vars(
(
card-border-color: $border,
card-cap-bg: $background,
card-cap-color: $color,
card-color: $color,
card-bg: tint-color($background, 5%)
)
);

Are there any updates on this feature?


Update coverage dependency

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.874%

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
429

Relevant Lines:
932

💛  - Coveralls


test: add postcss custom feature test case

Codecov Report

Merging #3591 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #3591   +/-   ##
=======================================
  Coverage   99.19%   99.19%
=======================================
  Files          18       18
  Lines        1114     1114
  Branches      300      300
=======================================
  Hits         1105     1105
  Misses          9        9

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a0ca257...61caa62. Read the comment docs.


Support WalletConnect

Does enKrypt plan to support WalletConnect?

Yes we have this planned however, we currently don't have it in the immediate road map


Update path to _config.yml in tutorial

Current version of tutorial shows path to _config.yml file in root directory. The config file is stored in the docs directory.
This PR fixes the path to the file in the tutorial.

You have to change the notebook, not just the HTML :) this project is written with nbdev, so the best route is to change the notebook that generates this page!

Thank you for the review. In this MR I update the notebook (.ipynb file), not the generated HTML. But I created new PR https://github.com/fastai/nbdev/pull/531, where both the notebook and HTML are updated.


The generics don't work with the type of a key value object

TypeScript Version:  2.4.0 / nightly (2.5.0-dev.201xxxxx)
Code
// A *self-contained* demonstration of the problem follows...
function test<T extends string | number>(a: T): {[key: T]: boolean} {
    const map: { [key: T]: boolean } = {};
    return map;
}

Expected behavior:
The type system should not complain that the key type of map should be string or number.
Actual behavior:
It gives error that the key of map should be string or number even though T is string or number

Possibly related
type Key = string | number;

const dict: { [index: Key]: any } = {};

duplicate of https://github.com/Microsoft/TypeScript/issues/16760

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


Split Bill - Number keeps on changing its initials

Details

Selected and unselected users  are displayed differently on the split screen found by a phone number
Fixed Issues

$ https://github.com/Expensify/App/issues/27557
PROPOSAL: https://github.com/Expensify/App/issues/27557#issuecomment-1731150813
Tests

Open Split screen
Enter some amount
In email or phone enter "+919999676352"
Select/Unselect the user suggested below

[x] Verify that no errors appear in the JS console

Offline tests

Open Split screen
Enter some amount
In email or phone enter "+919999676352"
Select/Unselect the user suggested below

QA Steps

Open Split screen
Enter some amount
In email or phone enter "+919999676352"
Select/Unselect the user suggested below

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/68128028/eb0eb7c7-b7af-4da5-b701-f6ad1f97f335

Mobile Web - Chrome
https://github.com/Expensify/App/assets/68128028/2b8f1c3d-1ebd-4bbf-b9bb-2f53d5a4163b

Mobile Web - Safari
https://github.com/Expensify/App/assets/68128028/1081123c-262b-45c9-9b41-830d835d6f89

Desktop

https://github.com/Expensify/App/assets/68128028/12881ca2-adf3-47f3-9681-a07ea93694bc

iOS

https://github.com/Expensify/App/assets/68128028/5649f65b-cf86-4fda-9c54-f75d35cd0d57

Android

https://github.com/Expensify/App/assets/68128028/a9de9840-5a40-4595-b17d-6b37bd7cffd1

@robertKozik please test soon and then I'll merge assuming it tests well.

Will test it today 👀

Reviewer Checklist

[X] I have verified the author checklist is complete (all boxes are checked off).
[X] I verified the correct issue is linked in the ### Fixed Issues section above
[X] I verified testing steps are clear and they cover the changes made in this PR

[X] I verified the steps for local testing are in the Tests section
[X] I verified the steps for Staging and/or Production testing are in the QA steps section
[X] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[X] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/neil-marcellini in version: 1.3.76-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.76-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/neil-marcellini in version: 1.3.77-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.77-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Enable x_host and x_proto by default through deprecated proxyfix?

When #1314 refactored ProxyFix, it aliased num_proxies to x_for only (as that was the only one which previously supported multiple proxies).
However in doing so it disabled forwarding of host and scheme, which were forwarded by default up to 0.14, which breaks systems relying on this.
I'd like to set x_host=1 and x_proto=1 in 0.15 IFF the user goes through the deprecated wrapper (werkzeug.contrib.fixers.ProxyFix) in order to restore the old behaviour, and maybe add a small warning about this behavioural change to the deprecation message (or possibly in one of the versionchanged blocks of the new version). Would that be acceptable / interesting or a waste of time?

Yeah, that sounds good.
I'm fine with adding x_proto=1 for both, not just for the deprecated wrapper, since we think it's fairly low impact on security.
x_host I'm less confident about going forward, as I'm not sure what the impact is if the client can control Host through X-Forwarded-Host when Nginx isn't configured to set it. Odoo probably has a better understanding of HTTP security than I do, so if you have a good argument for leaving it enabled, I'm interested in hearing it.


containerd fails on start: "failed to get metadata for stored sandbox"

Description
We observed one of our nodes failing to properly boot. The error is:
Oct 17 13:46:12 containerd[40100]: time="2024-10-17T13:46:12.015594695Z" level=info msg="Start recovering state"
Oct 17 13:46:12 containerd[40100]: time="2024-10-17T13:46:12.015677461Z" level=info msg=serving... address=/run/containerd/containerd.sock.ttrpc
Oct 17 13:46:12 containerd[40100]: time="2024-10-17T13:46:12.016081023Z" level=info msg=serving... address=/run/containerd/containerd.sock
Oct 17 13:46:12 containerd[40100]: time="2024-10-17T13:46:12.062483416Z" level=fatal msg="Failed to run CRI service" error="failed to recover state: failed to get metadata for stored sandbox \"4ea7054a2b00694197040ac0c59ac2552624824adf3df31c292243992c72bfc9\": not found"

Workaround: rm /var/lib/containerd/io.containerd.metadata.v1.bolt/meta.db then restarting containerd. I guess that this might leave some garbage in /var/lib/containerd/ so you could also consider reimaging the node.
Steps to reproduce the issue
We've not yet reproduced the issue, it's a one-off so far. I've seen two other references: https://github.com/siderolabs/talos/issues/9496 (from 5 days ago) and https://github.com/containerd/containerd/discussions/10236 (from May 16th).
I'll update here if I see this again or if I can reproduce it.
Describe the results you received and expected
I expect that containerd is able to recover state without failing, ignoring/deleting any invalid entries in the metadata.
What version of containerd are you using?
2.0.0-rc.4
Any other relevant information
# runc --version
runc version 1.1.14
commit: 2c9f5602f0ba3d9da1c2596322dfc4e156844890
spec: 1.0.2-dev
go: go1.23.0
libseccomp: 2.5.5
# uname -a
Linux hostname 6.6.42-rt37 #1 SMP PREEMPT_RT Thu Sep 12 11:03:43 CEST 2024 x86_64 AMD Ryzen 5 5600X 6-Core Processor AuthenticAMD GNU/Linux

Show configuration if it is related to CRI plugin.
No response

@samuelkarp FYI  https://github.com/containerd/containerd/blame/main/internal/cri/server/restart.go#L107-L111  fatal error here is causing a failure to restart the CRI plugin.. Need to figure out a way to clean and continue from this state.

I have also run across this today and it has completely halted a single node cluster. I guess I'll have to try the workaround of just removing the DB.

Unfortunately removing the DB breaks other things.
<IP_ADDRESS>   etcd         Failed    ?        2m38s ago     Failed to run pre stage: failed to pull image "gcr.io/etcd-development/etcd:v3.5.16": 1 error(s) occurred:
                failed to pull image "gcr.io/etcd-development/etcd:v3.5.16": failed to prepare extraction snapshot "extract-876211769-cOcg sha256:c79c5237324588ec7d6e94a0d95d226cde0da9d18be6598a9cf5aaa474ee6820": parent snapshot sha256:35c6024c94ed6d39380f78210d243b7422dcdb825146e51c69e6f4b8ff1f9277 does not exist: not found

The solution is to just delete the entire /var/lib/containerd directory.


Target suitable agents for deploy step

Occasionally a deploy would fail because the MacStadium agent would pick up the job, and it isn't configured with Heroku deploy credentials.

resolves https://buildkite.com/thelookoutway/heroku-drain-datadog/builds/181#0184164c-5c3e-4731-9aa6-eee82756a06f

Sorry for the delay. This week has been busy!

No worries, this wasn't urgent. This repository doesn't see much action.


[minizip] build failure (assimp)

Host Environment

OS: Windows
Compiler: VS2019

To Reproduce
Steps to reproduce the behavior:
./vcpkg install assimp:x64-windows
Failure logs
The following packages will be built and installed:
    assimp[core]:x64-windows -> 5.0.1#4
  * minizip[core]:x64-windows -> 1.2.11#7
  * poly2tri[core]:x64-windows -> 2020-07-21
  * polyclipping[core]:x64-windows -> 6.4.2#6
  * rapidjson[core]:x64-windows -> 2020-09-14
  * stb[core]:x64-windows -> 2020-09-14
  * utfcpp[core]:x64-windows -> 3.1.2#1
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...

-- Note: minizip only supports static library linkage. Building static library.
-- Using cached C:/dev/vcpkg/downloads/madler-zlib-v1.2.11.tar.gz
-- Cleaning sources at C:/dev/vcpkg/buildtrees/minizip/src/v1.2.11-a5237a6809.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/dev/vcpkg/downloads/madler-zlib-v1.2.11.tar.gz
-- Applying patch 0001-remove-ifndef-NOUNCRYPT.patch
-- Applying patch 0002-add-declaration-for-mkdir.patch
-- Using source at C:/dev/vcpkg/buildtrees/minizip/src/v1.2.11-a5237a6809.clean
-- Configuring x64-windows
-- Building x64-windows-dbg
-- Building x64-windows-rel
CMake Error at scripts/cmake/vcpkg_copy_tool_dependencies.cmake:26 (message):
  Could not find PowerShell Core; please open an issue to report this.
Call Stack (most recent call first):
  ports/minizip/portfile.cmake:35 (vcpkg_copy_tool_dependencies)
  scripts/ports.cmake:136 (include)

Error: Building package minizip:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: minizip:x64-windows
  Vcpkg version: 2020.11.12--unknownhash

Any help is greatly appreciated, just built a new pc and need to get back to work. In the log it says it cannot find powershell core, but I do indeed have powershell core installed.

@TigerCipher  Can you please make sure your vcpkg is latest?
And which powershell version is downloaded in VCPKG_ROOT/downloads/tools/powershell-core-***-windows?

@TigerCipher  Can you please make sure your vcpkg is latest?
And which powershell version is downloaded in VCPKG_ROOT/downloads/tools/powershell-core-***-windows?

@TigerCipher Can you please make sure your vcpkg is latest?
And which powershell version is downloaded in VCPKG_ROOT/downloads/tools/powershell-core-***-windows?

I just installed vcpkg today, and just for sanity I tried updating it anyway. So its definitely the latest version. There is no powershell folder in downloads/tools, but I have Powershell Core 7.1.0 on my system

@TigerCipher Can you please make sure your vcpkg is latest?
And which powershell version is downloaded in VCPKG_ROOT/downloads/tools/powershell-core-***-windows?

I just installed vcpkg today, and just for sanity I tried updating it anyway. So its definitely the latest version. There is no powershell folder in downloads/tools, but I have Powershell Core 7.1.0 on my system

cc @BillyONeal

cc @BillyONeal

@TigerCipher What do you get for:
.\vcpkg.exe fetch powershell-core

Thanks!

@TigerCipher What do you get for:
.\vcpkg.exe fetch powershell-core

Thanks!

(And what is your %PATH% ?)

(And what is your %PATH% ?)

@TigerCipher What do you get for:
.\vcpkg.exe fetch powershell-core

Thanks!

This actually said I didn't have a suitable version and it downloaded a portable version to the tools folder. My path is:
PATH=C:\Windows\system32;
C:\Windows;
C:\Windows\System32\Wbem;
C:\Windows\System32\WindowsPowerShell\v1.0\;
C:\Windows\System32\OpenSSH\;
C:\Program Files (x86)\NVIDIA Corporation\PhysX\Common;
C:\Program Files\NVIDIA Corporation\NVIDIA NvDLISR;
C:\Program Files\Git\cmd;
C:\Program Files\dotnet\;
C:\Program Files\PowerShell\7\;
C:\Users\mattr\AppData\Local\Microsoft\WindowsApps;
C:\Users\mattr\.dotnet\tools

So I am able to install the package I was trying to install now since it downloaded a portable version. Not sure why it doesn't recognize the powershell in my path though

@TigerCipher What do you get for:
.\vcpkg.exe fetch powershell-core

Thanks!

This actually said I didn't have a suitable version and it downloaded a portable version to the tools folder. My path is:
PATH=C:\Windows\system32;
C:\Windows;
C:\Windows\System32\Wbem;
C:\Windows\System32\WindowsPowerShell\v1.0\;
C:\Windows\System32\OpenSSH\;
C:\Program Files (x86)\NVIDIA Corporation\PhysX\Common;
C:\Program Files\NVIDIA Corporation\NVIDIA NvDLISR;
C:\Program Files\Git\cmd;
C:\Program Files\dotnet\;
C:\Program Files\PowerShell\7\;
C:\Users\mattr\AppData\Local\Microsoft\WindowsApps;
C:\Users\mattr\.dotnet\tools

So I am able to install the package I was trying to install now since it downloaded a portable version. Not sure why it doesn't recognize the powershell in my path though

That sounds something like an old vcpkg.exe which was missing the bit to acquire pwsh being used with new ports. Any chance that was copied from somewhere else or did you just bootstrap recently?
If you delete the pwsh it just downloaded in your above attempt, rerun bootstrap, and re attempt to install does that fix it?

That sounds something like an old vcpkg.exe which was missing the bit to acquire pwsh being used with new ports. Any chance that was copied from somewhere else or did you just bootstrap recently?
If you delete the pwsh it just downloaded in your above attempt, rerun bootstrap, and re attempt to install does that fix it?

@TigerCipher, does this still be a problem for you?