[Web] App crashes when trying to print a string contains � character

I got http string response containing "�" testing on chrome debugger, and flutter framework crashes.
Steps to Reproduce

Declare a string variable like below.
const wrongchar = "�";
print(wrongchar);
run app

Expected results:
console prints "�" or �
Actual results:
Got crash
I also tried convert string with utf8.decode(wrongchar), but it resulted same.

Code sample

const wrongchar = "�";
print(wrongchar);

Logs

Finished with error: Bad UTF-8 encoding found while decoding string: �
. The Flutter team would greatly appreciate if you could file a bug explaining exactly what you were doing when this happened:
https://github.com/flutter/flutter/issues/new/choose
The source bytes were:
[239, 191, 189, 10]

Analyzing myapp...

   ...
   Other file analysis...
   ...
   info - Avoid `print` calls in production code - lib\nav\sub\third_page.dart:36:5 - avoid_print

35 issues found. (ran in 1.6s)\

[√] Flutter (Channel stable, 2.10.0, on Microsoft Windows [Version 10.0.19043.1526], locale ko-KR)
    • Flutter version 2.10.0 at D:\flutter_windows_2.10.0-stable\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5f105a6ca7 (12 days ago), 2022-02-01 14:15:42 -0800
    • Engine revision 776efd2034
    • Dart version 2.16.0
    • DevTools version 2.9.2

[√] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at C:\Users\ys\AppData\Local\Android\Sdk
    • Platform android-32, build-tools 32.1.0-rc1
    • ANDROID_HOME = C:\Users\ys\AppData\Local\Android\Sdk
    • Java binary at: C:\AndroidStudio\android-studio-211\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[X] Visual Studio - develop for Windows
    X Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[√] Android Studio
    • Android Studio at C:\AndroidStudio\android-studio-211
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = C:\AndroidStudio\android-studio-211
    • Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)

[√] IntelliJ IDEA Community Edition (version 2021.1)
    • IntelliJ at D:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2020.1.1
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin version 211.7798

[√] VS Code (version 1.55.0)
    • VS Code at C:\Users\ys\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension can be installed from:
       https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[√] Connected device (2 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19043.1526]
    • Chrome (web)      • chrome  • web-javascript • unknown

[√] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

@maheshmnj  Thank you for your quick reply.
I checked it on my android device, it prints the character without error.
As I know flutter supports web-app development. I added simple index.html guessing it is web-app development project, I met crash again.
Text widget can display �, only print crashes.

Do you think this is due to my chrome issue? or something else?
Please share your opinion.
If Flutter development team assume that this is not a critical issue, but I wanted to leave it for someone who will get content including the the character from web resources, struggling with such crash on chrome debugging system.

My Chrome version: Version 98.0.4758.82(Official build) (64 bit)

main.dart

import 'package:flutter/material.dart';

void main() => runApp(Hummingbird());

class Hummingbird extends StatelessWidget {

  @override
  Widget build(BuildContext context) {

    return MaterialApp(
      title: 'Hummingbird - Flutter For Web',
      debugShowCheckedModeBanner: false,
      theme: ThemeData(
        primarySwatch: Colors.teal,
      ),
      home: MyWebsite(),
    );
  }
}

class MyWebsite extends StatelessWidget {

  @override
  Widget build(BuildContext context) {

    const wrongchar = "�";
    print(wrongchar); // This cause crash

    return Scaffold(
      appBar: AppBar(
        title: Text(wrongchar), // This worked without any error.
        centerTitle: true,
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Container(
                width: 200.0,
                height: 200.0,
                decoration: BoxDecoration(
                    shape: BoxShape.circle,
                    image: DecorationImage(
                        fit: BoxFit.cover,
                        image: new NetworkImage(
                          "https://images.unsplash.com/photo-1520638023360-6def43369781?ixlib=rb-1.2.1&ixid=eyJhcHBfaWQiOjEyMDd9&auto=format&fit=crop&w=500&q=60",
                        )))),
            Padding(
              padding: const EdgeInsets.all(20.0),
              child: Text(
                'Welcome to Flutter For Web',
                style: TextStyle(fontSize: 24.0, fontWeight: FontWeight.bold),
              ),
            ),
          ],
        ),
      ),
    );
  }
}

index.html

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <meta name="test-meta" content="test-meta-content">
    <meta name="description" content="A simple implementation of Flutter for Web.">
    <script defer src="main.dart.js" type="application/javascript"></script>
</head>
<body>
</body>
</html>

Crash log

D:\Sandbox\13_Flutter\myapp>flutter run -d chrome
Launching lib\main.dart on Chrome in debug mode...
Waiting for connection from debug service on Chrome...             16.4s
This app is linked to the debug service: ws://<IP_ADDRESS>:10444/WUak-HNVlJw=/ws
Debug service listening on ws://<IP_ADDRESS>:10444/WUak-HNVlJw=/ws

 Running with sound null safety

  To hot restart changes while running, press "r" or "R".
For a more detailed help message, press "h". To quit, press "q".

An Observatory debugger and profiler on Chrome is available at: http://<IP_ADDRESS>:10444/WUak-HNVlJw=
Bad UTF-8 encoding found while decoding string: �
. The Flutter team would greatly appreciate if you could file a bug explaining exactly what you were doing when this happened:
https://github.com/flutter/flutter/issues/new/choose
The source bytes were:
[239, 191, 189, 10]

Thanks for the update @yuseok, I am able to reproduce the crash on stable and the master channel, when the trying to print "�" chracter web app crashed and works fine on Windows Desktop and Android. This crash happens  on  macOS Web too.

code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    const wrongchar = "�";
    print(wrongchar);
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headline4,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ),
    );
  }
}

logs
PS C:\Users\mahesh\Desktop\mysample> flutterm run -d chrome
Launching lib\main.dart on Chrome in debug mode...
Waiting for connection from debug service on Chrome...             32.2s
This app is linked to the debug service: ws://<IP_ADDRESS>:61530/kRmuyGNvEHY=/ws
Debug service listening on ws://<IP_ADDRESS>:61530/kRmuyGNvEHY=/ws

 Running with sound null safety

  To hot restart changes while running, press "r" or "R".
For a more detailed help message, press "h". To quit, press "q".

An Observatory debugger and profiler on Chrome is available at: http://<IP_ADDRESS>:61530/kRmuyGNvEHY=Bad UTF-8 encoding found while decoding string: �
. The Flutter team would greatly appreciate if you could file a bug explaining exactly what you
were doing when this happened:
https://github.com/flutter/flutter/issues/new/choose
The source bytes were:
[239, 191, 189, 10]

flutter doctor -v
[√] Flutter (Channel stable, 2.10.1, on Microsoft Windows [Version 10.0.19044.1466], locale
    en-US)
    • Flutter version 2.10.1 at C:\flutter_sdk\stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db747aa133 (5 days ago), 2022-02-09 13:57:35 -0600
    • Engine revision ab46186b24
    • Dart version 2.16.1
    • DevTools version 2.9.2

[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\mahesh\AppData\Local\Android\sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.4)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.11.31727.386
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2020.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[√] VS Code (version 1.63.2)
    • VS Code at C:\Users\mahesh\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.34.0

[√] Connected device (5 available)
    • Redmi K20 Pro (mobile)       • <IP_ADDRESS>:5555 • android-arm64  • Android 11 (API 30)
    • sdk gphone64 x86 64 (mobile) • emulator-5554    • android-x64    • Android 12 (API 31)
      (emulator)
    • Windows (desktop)            • windows          • windows-x64    • Microsoft Windows [Version      10.0.19044.1466]
    • Chrome (web)                 • chrome           • web-javascript • Google Chrome 98.0.4758.82    • Edge (web)                   • edge             • web-javascript • Microsoft Edge
      97.0.1072.76

[√] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[√] Flutter (Channel master, 2.11.0-0.0.pre.554, on Microsoft Windows [Version 10.0.19044.1526],
    locale en-US)
    • Flutter version 2.11.0-0.0.pre.554 at C:\flutter_sdk\master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 894ef49c7c (2 hours ago), 2022-02-14 22:15:17 -0500
    • Engine revision 8714a2448f
    • Dart version 2.17.0 (build 2.17.0-105.0.dev)
    • DevTools version 2.10.0

[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\mahesh\AppData\Local\Android\sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.4)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.11.31727.386
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2020.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[√] VS Code (version 1.63.2)
    • VS Code at C:\Users\mahesh\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.34.0

[√] Connected device (4 available)
    • Windows (desktop)       • windows • windows-x64     • Microsoft Windows [Version
      10.0.19044.1526]
    • Windows (UWP) (desktop) • winuwp  • windows-uwp-x64 •
    • Chrome (web)            • chrome  • web-javascript  • Google Chrome 98.0.4758.102
    • Edge (web)              • edge    • web-javascript  • Microsoft Edge 97.0.1072.76

[√] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

cc: @yjbanov

flutter_tools thinks that the "�" character indicates an error in UTF-8 decoding and is not aware that the app is trying to print that character intentionally:
https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/convert.dart#L53

flutter_tools thinks that the "�" character indicates an error in UTF-8 decoding and is not aware that the app is trying to print that character intentionally: https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/convert.dart#L53

I think it is needed to clarify its cause in error output. Please see my PR: https://github.com/flutter/flutter/pull/98553


Transaction History Conversions

The Outcome for a scalar market comes back as an on contract style value and we need to make sure it's converted to a display value. An example would be a scalar where 3,000 is the value we expect as an outcome name but instead we receive<PHONE_NUMBER>000000000. Alternatively we could just use the scalar description (Degrees etc).
We also get on Contract values back for share amounts and totals, very similar to the above issue.
Finally we are seeing a lot of these tagged as ETH but they should probably be looking at DAI values now or sending labels for DAI instead of ETH at the very least if the values are accurate to DAI already.

I can look at scalars.

status?

Looks like scalars and Dai show correctly now....
Looks like their are 3 issues we have:

We are with showing a number in Outcome for Initial Report
In the Total column for Claim Stake and Fees we show a long string (I think we are missing the decimal place)
Doesnt look like we show the trading Fee

fixed closing


Proxy video, just like images

Let the image proxy support videos too. The video element has a some source sub-elements, with the src indicating the URL. Everything else can work the same way.
The only real difference is that some videos might be bigger in size, the proxy should retry cleanly if the download is interrupted before the download concludes.

#615 is the same request, for audio.


jdom 1.0

Type: Missing
Summary:
jdom 1.0
Details:
ClearlyDefined license file is BSD-3-Clause
Maven indicates Apache but does not include a link and no info found in POM
Resolution:
BSD-3-Clause
Affected definitions:

jdom 1.0

@LandOfBliss The BSD-3-Clause license text has been modified to include a disclaimer. Perhaps this should be OTHER? @ariel11 Thoughts?

@AE49 I agree with OTHER.


#34741 Fix pseudo columns and pseudo tables highlighting

Dual table for Oracle.
Pseudo columns for Oracle https://docs.oracle.com/en/database/oracle/oracle-database/19/sqlrf/Pseudocolumns.html

ORA_ROWSCN
ROWID

Pseudo columns for SQLite https://www.sqlite.org/lang_createtable.html#rowid Would be nice to also test on old SQLite versions (version < 3.8, 3.8 < version <3.30, version > 3.30)

rowid
oid
rowid

Pseudo columns for PostgreSQL https://www.postgresql.org/docs/8.2/ddl-system-columns.html

oid
tableoid
xmin
cmin
xmax
cmax
ctid
and global variables
current_date
current_time
current_timestamp
localtime
localtimestamp
current_role
current_user
current_catalog
current_schema
session_user
system_user
user

Global variables for DB2

CLIENT_IPADDR
DATASLICEID

Please check pseudo columns visibility in different select query parts. If column is highlighted with red, then it's not available and the query can't run. If there is no red highlighting for pseudo attribute or global variable, then query should run. I mostly tested on plain select and subqueries. If you find that visibility depends on some other conditions, that are not taken in account yet, we can create a new issue for that. Unfortunately, documentation doesn't describe it well.
Global variables are always visible.

Simple select query check:

Dual tables

Pseudo columns, oracle

Pseudo columns for SQLite
SQLite 3.46.1

3.8.6

3.28.0

TBD

verified


Remove wireguard interfaces when deleting the cluster

Signed-off-by: Manuel Buil<EMAIL_ADDRESS>

Proposed Changes

This PR includes the deletion of the wireguard interfaces when destroying the cluster. Those interfaces are created when we choose flannel-backend: wireguard-native
Types of Changes

Bugfix
Verification

Deploy k3s with flannel-backend: wireguard-native and then destroy the cluster. You should see those interfaces disappear.
Note that flannel-wg-v6 only appears in dual-stack or ipv6-only
Testing

Linked Issues

https://github.com/k3s-io/k3s/issues/6053
User-Facing Change

Further Comments

Codecov Report

Merging #6055 (a6f5a4d) into master (ba62c79) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #6055   +/-   ##
======================================
  Coverage    9.72%   9.72%
======================================
  Files         138     138
  Lines       10060   10060
======================================
  Hits          978     978
  Misses       8883    8883
  Partials      199     199

Flag
Coverage Δ

unittests
9.72% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Use separate backoff schedule or poll interval for resolving managed resource references

What problem are you facing?

Currently if we are unable to resolve references, we return Requeue: true, causing us to enter backoff as configured for the managed resource reconciler. However, resolving references typically is an operation that is expected to fail early and potentially be resolved fairly quickly (i.e. we are typically waiting for another resource to be provisioned). This is inherently a different situation than when we encounter errors actually trying to provision the resource, and does not involve making external API calls. For this reason, we are frequently waiting much longer between reconciles than makes sense.
Example of where this may be happening: https://github.com/crossplane/provider-aws/issues/1062
How could Crossplane help solve your problem?

A quick short-term solution could be to just use the poll interval for the controller, but that is once again likely a much longer period than we want to wait for many reference resolution scenarios. An alternative solution would be to provide a separate backoff schedule for reference resolution which could potentially be exposed to users.
xref https://github.com/crossplane/crossplane/pull/2602 https://github.com/crossplane/crossplane/issues/2595 https://github.com/crossplane/crossplane/issues/2477 https://github.com/crossplane/crossplane-runtime/issues/253
cc @negz @muvaf

I'm wondering if it makes sense to have a "ready" poll interval and a "not-ready" poll interval, both for crossplane and providers.
When ready, a resource should generally be stable, and even an interval of 30 minutes might make sense in some setups and will not impact the user.
For newly provisioned managed resources and compositions (maybe identified by "not available/ready" and/or recently created), it might make sense to observe them fairly quickly, either for propagating ready status of the managed resource, resolving references or propagating data using patches. This could either be a low interval or with a backoff algorithm, like 5, 10, 30 seconds after creation.

prior art in flux:
https://fluxcd.io/flux/components/kustomize/kustomization/
  interval: 60m0s # detect drift and undo kubectl edits every hour
  timeout: 3m0s # give up waiting after three minutes
  retryInterval: 2m0s # retry every two minutes on apply or waiting failures


go: 1.14.1 -> 1.14.2

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/go/versions.
meta.description for go is: "The Go Programming language"
meta.homepage for go is: "http://golang.org/"
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.14.2 with grep in /nix/store/pdlxgwz8g5flf9ng92d0vz6rrvv64hcn-go-1.14.2

directory tree listing: https://gist.github.com/01cf90f96ee7814bb8ced0b6d3ffa250

du listing: https://gist.github.com/48ae41c060fb1e3805770e5e64fc9031

Rebuild report (if merged into master) (click to expand)

3220 total rebuild path(s)

841 package rebuild(s)

836 x86_64-linux rebuild(s)
816 i686-linux rebuild(s)
752 x86_64-darwin rebuild(s)
816 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
_3mux
act
aerc
age
alertmanager-bot
amass
amazon-ecr-credential-helper
aminal
antibody
aptly
archiver
argo
argocd
asmfmt
assh
atlantis
autospotting
awless
aws-env
aws-iam-authenticator
aws-okta
aws-rotate-key
aws-vault
awsweeper
azure-storage-azcopy
azure-vhd-utils
bazel-buildtools
bazel-gazelle
bazel-remote
bazel-watcher
bazelisk
berglas
bettercap
blockbook
blsd
boohu
boringssl
bosun
boulder
brig
brillo
brook
browserpass
browsh
btops
buildah
buildifier
buildkit
buildkite-agent
buildkite-agent3

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/pdlxgwz8g5flf9ng92d0vz6rrvv64hcn-go-1.14.2 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A go https://github.com/r-ryantm/nixpkgs/archive/d5dd42ca504f2c93e1dbce24b50841530047a5d1.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/pdlxgwz8g5flf9ng92d0vz6rrvv64hcn-go-1.14.2
ls -la /nix/store/pdlxgwz8g5flf9ng92d0vz6rrvv64hcn-go-1.14.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the change, simply because they are already broken on the target branch.

Maintainer pings
cc @cstrahan @orivej @mic92 @rvolosatovs @kalbasit @Frostman for testing.

Duplicate of https://github.com/NixOS/nixpkgs/pull/84801


Pin docker dep and run 'go get -u -t ./...'

Use this PR instead of #123

@joshuatcasey I have an open PR to roll out a Syft bump. There's no need to pin anything here since that change will hopefully come out soon.

@sophiewigmore Good to know. I'll leave this PR here but block it, just so I have a reminder to come back to it.
Blocked on: paketo-buildpacks/packit#440

I deleted the branch and closed the PR as it's simpler to let the automation recreate this against the changes on main.


feat: create catalog of uploaded charts

fixes https://github.com/kokuwaio/helm-maven-plugin/issues/327

This pr is unchanged since last week. Please notify me when i should review this pr. Thanks!

This pr is unchanged since last week. Please notify me when i should review this pr. Thanks!

Sure. I will let you know. Thanks.

@sschnabe
Please feel free to review now. Since, I am on vacation, there will be delay in fixing any review comments. Thanks.

Can someone please review the PR? If you have any questions/comments already, please let me know.
It would be really helpful if the feature is available by the end of the month.

@kumadee  Thx, i'll release a new version next week after #338 is merged.

Released with 6.12.0


Update Amaranth version

This PR updates Amaranth to the latest version, in preparation for using amaranth.lib.data in Transactron. Some fixes to typing and stubs are also included.
Stubs for enums probably need to be changed because of Amaranth's RFC 31, but current stubs are good enough for now.

Issue filled.
https://github.com/amaranth-lang/amaranth/issues/976


Dockerfile: add version support to dockerfiles in order to use nightly, stable, latest or versioned images

What is the expected enhancement?
Add version support to dockerfiles in order to use nightly, stable, latest or versioned images

By @psschwei
assuming you meant the compose file, we could do something like:
services:
  jupyter:
    container_name: qs-jupyter
    image: qiskit/quantum-serverless-notebook:${VERSION:-0.0.6}-py39
    ...

and then if you wanted to run nightly do something like VERSION=nightly docker compose up

Something like:
Dockerfile:
ARG TAG=0.0.6
FROM image:$TAG
...

and then to build the nightly build, run docker build --build-arg TAG=nightly ...

As a user I just want to run without building images
git clone ...
cd ...
docker-compose up --file=compose-latest

@psschwei do you know if we can do confitional inside docker-compose file?
If I want bare minimal docker-compose setup with only gateway, keycloak and ray. Cuz right now it fetches like 100 services :) To run something like
SERVICES=minimal docker compose up

You can specify the ones you want to run, so docker compose up gateway keycloak ray-head should get you just those three (assuming I have the names right)

docker compose up gateway keycloak ray-head

indeed!


drivers: eth: gmac: Fix MAC address info log

Each MAC byte should be printed with the %02x format.
Signed-off-by: Tomasz Gorochowik<EMAIL_ADDRESS>

Codecov Report

Merging #15078 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #15078      +/-   ##
==========================================
- Coverage   52.93%   52.92%   -0.02%
==========================================
  Files         309      309
  Lines       45268    45268
  Branches    10451    10451
==========================================
- Hits        23961    23956       -5
- Misses      16542    16544       +2
- Partials     4765     4768       +3

Impacted Files
Coverage Δ

subsys/testsuite/ztest/src/ztest.c
68.59% <0%> (-4.14%)
:arrow_down:

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 766edd4...8c5fb37. Read the comment docs.


Restkit Oauth ignores port numbers when generating signature

If you're developing against a local oauth provider, say a rails process at http://localhost:3000, the signature being generated by the restkit will drop the "3000" when generating its signature in GCOAuth.m, thus making the signature invalid when being compared against the server's for an Oauth 1.0 signature.

Seems this is a product of the GCOAuth forked from tweetdeck which expects the url in components, excluding the port number.

@imackinn Did you ever come up with a workaround for this? I'm running into the exact same problem (Django server at localhost:8000), and it's dropping the port.

I have a pull request from my fork. There's also an "uppercase/lowercase" issue. There's a question of whether it works in the general case, which I haven't had time to investigate yet, but it solves the localhost problem. It is a bit of a hack on GCOAuth though.

@imackinn I had just been in the process of discovering the same workaround. It looks like the URL builder also drops any trailing slash, so I added that in to mine as well (so now it's @"%@://%@:%@%@/", ie).

Issue #533 is similarly about the url builder from the uppercase. Looks like there seems to be a few around the GCOauth url builder. Guess it was more twitter-centric.


Must pass exactly one of edge_set_name, node_set_name

Hi,
raise ValueError(
"Must pass exactly one of edge_set_name, node_set_name "
"for receiver_tag CONTEXT.")
This means only one type of nodes can be conv at a time? So need to conv many times for multiple types of nodes?

The indices of nodes  in a edge set may be from different node sets?

Maybe classify a graph by one type of node.


Hotfix/troubleshooting tmp err

spaces trimmed for registration and login forms on amigo 91f3b73
duplicate getevents deleted 52d8786
check added here 795e83f
troubleshooting doc introduced with initial information bb44314

I have tested this branch in local and seem that strings.TrimSpace dosen't work as expected. The problem of white space is still here. You should find another way to solve it i guess

Instead it is working, TrimSpace does not mean that it will remove spaces between words, it will gonna remove the spaces before and after the word.  Here you go;
https://play.golang.org/p/A5Z75hurO_4
@gianmarcomennecozzi
The problem was with that particular topic is that for instance I have an account called "mrturkmen", if I would like to register with "mrturkmen ", I was able to register, however, after TrimSpace, it is not possible. That was the our problem regarding to team names.
However, I realized that when we are saving team into teamstore, we do not check usernames, we only rely on emails that's why it is possible to create multiple teams with same name. I think, the problem is more than TrimSpace.

Okay then i think its good to remove spaces also from the inserted uername and check if that username has been already choosen.

Okay then i think its good to remove spaces also from the inserted uername and check if that username has been already choosen.

I have already added it, check [this line] (https://github.com/aau-network-security/haaukins/blob/21656b82cb5978379329108e003082b07b40b48d/store/team.go#L88).


[Tizen] Need to check PlatformManager Init / Shutdown function in detail

This issue is for handling the potential issue for PlatformManager's Init() and Shutdown() API.
We found the deadlock issue when the application calls Shutdown twice. Because the deadlock issue is very critical. So to release the hotfix patchset, we added the prevent logic for deadlock.
https://github.com/project-chip/connectedhomeip/pull/28094
But still remain the potential issues, so need to handle it using this issue.

https://github.com/project-chip/connectedhomeip/pull/28094#discussion_r1268937576
I would ask why the Shutdown() is called twice? If that's a normal behavior, then you will have to move this check at the beginning of this function (above GenericPlatformManagerImpl_POSIX::_Shutdown()), because calling GenericPlatformManagerImpl_POSIX::_Shutdown() twice will cause double-free.
However, if we need to protect against calling Shutdown() twice in a row, what about Init()? It also can not be called twice (without calling Shutdown in between). And this issue is not Tizen specific. IMHO, the rule applies to almost all Init/Shutdown calls.
Slack's converstions.
Arkadiusz Bokowy
1일 전
Hi, I've got a question regarding one PR for Tizen: https://github.com/project-chip/connectedhomeip/pull/28094
We've got an issue where the Shutdown() is called twice, and I'd like to ask someone who is more familiar with the overall design, whether calling Init()/Shutdown() twice (or more) in a row is in general allowed and should be handled by the platform code? Or maybe if one of these functions is called more than once in a row is a sign of an issue somewhere else? I'm asking, because there are other platforms where Init() and Shutdown() are not idempotent.
#28094 [Tizen] Fix the deadlock issue during shutdown the system layer
When the app calls PlatformManager's Shutdown API twice, the function tries to join the abnormal main loop's thread. Adds the prevent function to avoid this deadlock issue.
Labels
platform, review - pending, tizen
https://github.com/[project-chip/connectedhomeip](https://github.com/project-chip/connectedhomeip)|project-chip/connectedhomeipproject-chip/connectedhomeip | 어제 오후 1:42 | 봇이 추가한 GitHub
5개의 댓글
Boris Zbarsky
16시간 전
I would think double-shutdown is generally an indication of an issue....
Boris Zbarsky
16시간 전
But depends on how the API is documented.
Arkadiusz Bokowy
15시간 전
Yes, we can blame the documentation :웃음: but I'll like to know what is the grand design for using Init()'s instead of using constructors/destructors, and whether Init()/Shutdown() should behave like ones. As for now we are OK with a quick fix of making Shutdown() idempotent, but maybe later it should be fixed in a more general/platform agnostic way.
Boris Zbarsky
5시간 전
Oh, PlatformManager shutdown is definitely not OK to call twice....
Arkadiusz Bokowy
1시간 전
OK, thanks

When I added the simulation codes called PlatformManager's Init() twice, the platform's impl function is not called twice.
In PlatformManager.h file, already added the prevent code.
https://github.com/project-chip/connectedhomeip/blob/master/src/include/platform/PlatformManager.h#L366
Same with this, we can add the preventing logic in PlatformManager.h's inline function. And it will be the generic solution for all platforms. But the change will effect on all platform's operation. So I am not sure, it will be good to raise PR in my side.
diff --git a/src/include/platform/PlatformManager.h b/src/include/platform/PlatformManager.h
index 41334a465b..6d5b682956 100644
--- a/src/include/platform/PlatformManager.h
+++ b/src/include/platform/PlatformManager.h
@@ -442,6 +442,11 @@ inline CHIP_ERROR PlatformManager::StopEventLoopTask()
*/
inline void PlatformManager::Shutdown()
{

if (!mInitialized)
{

   return;

}
static_cast<ImplClass *>(this)->_Shutdown();
mInitialized = false;
}


[Book] Wrong action name in comment

https://github.com/gtk-rs/gtk4-rs/blob/ed840ef06f0aa9a2e7412d4fdd9cac53cfbc976c/book/listings/actions/3/main.rs#L68
This should be win.count  I think. @Hofer-Julian
Btw. well done with the chapter :)

Thanks @AaronErhardt :)
Wanna open a PR?


Is this plugin currently working?

I can't get the simplest example to work, something wrong with the css?
https://codepen.io/jmichaelterenin/pen/JBOQdq

This code form codepen, when copied to my local environment, doesn't work, the plugin acts as if it's broken. I've never experienced this before, any ideas as to what the problem would be?

Never mind, apparently not having rel="stylesheet" in the link element is an issue


DBRP server implementation does not match swagger specification

See: https://docs.influxdata.com/influxdb/v2.0/api/#operation/PostDBRP
https://github.com/influxdata/influxdb/blob/d93560fdc85f21d8a4720a7882698f38c5be28b2/dbrp/http_server_dbrp.go#L58-L65
The API docs states:

org
orgID
bucketID

But we shorten organization to org, and we use camel-case in the docs / API spec.
Since the docs are the shared source of truth, we should update the implementation.

@GeorgeMac I've seen a few people in the community Slack posting curl examples for using this API, I'm wary about changing the implementation and breaking them. It looks like IDPE is in the same situation (swagger here, and struct here). If Swagger is just used for documentation (no codegen), I'd feel better about fixing it to match the current behavior.

Some more data points / bugs:

The generated API docs show both org and orgID as both required, when (I believe) it's really one-or-the-other. Code here.
The DBRP model definition in the API spec is reused for the list endpoint. The model used by the implementation of that endpoint matches the implementation of the create endpoint, so the mismatch occurs there as well:
https://github.com/influxdata/influxdb/blob/d93560fdc85f21d8a4720a7882698f38c5be28b2/dbrp_mapping.go#L28-L38


Updated autogenerate script + index, map, Toc + Tanya Janca added

Updated autogenerate script with banner, filter, new image.
regenerate the index, index.html, Map, toc files
Added Tanya Janca profile

:warning: Validation status: warnings

File
Status
Preview URL
Details

advocates/index.html.yml
:warning:Warning
View
Details

advocates/index.yml
:warning:Warning
View
Details

advocates/map.yml
:warning:Warning
View
Details

advocates/tanya-janca.yml
:warning:Warning
View
Details

FromYmlToTOC-INDEX.ps1
:white_check_mark:Succeeded

advocates/media/profiles/tanya-janca.png
:white_check_mark:Succeeded
View

advocates/toc.yml
:white_check_mark:Succeeded
View

advocates/index.html.yml

[Warning] the value ProfileList is undefined for metadata pagetype

advocates/index.yml

[Warning] the value ProfileList is undefined for metadata pagetype

advocates/map.yml

[Warning] the value ProfileList is undefined for metadata pagetype

advocates/tanya-janca.yml

[Warning] the value Profile is undefined for metadata pagetype

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


K8SPS-43-: Log async replication not ready

CHANGE DESCRIPTION
Problem:
Improve logs.
CHECKLIST
Jira

[x] Is the Jira ticket created and referenced properly?
[x] Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
[x] Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

[x] Is an E2E test/test case added for the new feature/change?
[x] Are unit tests added where appropriate?

Config/Logging/Testability

[x] Are all needed new/changed options added to default YAML files?
[x] Did we add proper logging messages for operator actions?
[x] Did we ensure compatibility with the previous version or cluster upgrade process?
[x] Does the change support oldest and newest supported PS version?
[x] Does the change support oldest and newest supported Kubernetes version?

Test name
Status

async-ignore-annotations
passed

auto-config
passed

config
passed

config-router
passed

demand-backup
passed

gr-demand-backup
passed

gr-demand-backup-haproxy
passed

gr-finalizer
passed

gr-haproxy
passed

gr-ignore-annotations
passed

gr-init-deploy
passed

gr-one-pod
passed

gr-recreate
failure

gr-scaling
passed

gr-security-context
passed

gr-self-healing
passed

gr-tls-cert-manager
passed

gr-users
passed

haproxy
passed

init-deploy
passed

limits
passed

monitoring
passed

one-pod
failure

operator-self-healing
passed

recreate
passed

scaling
passed

service-per-pod
passed

sidecars
passed

smart-update
passed

tls-cert-manager
passed

users
passed

version-service
passed

We run 32 out of 32

commit: https://github.com/percona/percona-server-mysql-operator/pull/688/commits/0308c338b1c926a915f66f6e49de65760df98db4
image: perconalab/percona-server-mysql-operator:PR-688-0308c338

Test name
Status

async-ignore-annotations
passed

auto-config
passed

config
passed

config-router
passed

demand-backup
passed

gr-demand-backup
passed

gr-demand-backup-haproxy
passed

gr-finalizer
passed

gr-haproxy
passed

gr-ignore-annotations
passed

gr-init-deploy
passed

gr-one-pod
passed

gr-recreate
passed

gr-scaling
passed

gr-security-context
passed

gr-self-healing
passed

gr-tls-cert-manager
passed

gr-users
passed

haproxy
passed

init-deploy
passed

limits
passed

monitoring
passed

one-pod
passed

operator-self-healing
passed

recreate
passed

scaling
passed

service-per-pod
passed

sidecars
passed

smart-update
passed

tls-cert-manager
passed

users
passed

version-service
passed

We run 32 out of 32

commit: https://github.com/percona/percona-server-mysql-operator/pull/688/commits/0308c338b1c926a915f66f6e49de65760df98db4
image: perconalab/percona-server-mysql-operator:PR-688-0308c338


test/k8sT: test default-deny ingress-egress policy

Add an end-to-end test for testing the default-deny ingress and egress policy for Kubernetes. This was added for completeness' sake to show that we are testing all default deny combinations of policy for Kubernetes.
Signed-off by: Ian Vernon<EMAIL_ADDRESS>Related-to: #2337

test-me-please

test-me-please

test-me-please

test-missed-k8s

test-me-please

The missed K8s tests were hit by the timeout issue fixed by https://github.com/cilium/cilium/pull/3837 .


update front page of SDM4 in R website

The front page is a bit stark at present (see https://nhorton.people.amherst.edu/sdm4/).  @sarahmcdonald20 can you please work to add some additional material about these materials, the mosaic project, and a pointer to the book website at Pearson?

Hi Nick,
I've made some changes to the front page and wanted to preview them on the website. What was the code to update the changes into the _site folder? Thanks! @nicholasjhorton

See https://rmarkdown.rstudio.com/rmarkdown_websites.html
You'll probably want to spend some time reviewing the documentation and seeing some of the bells and whistles that are supported.
It might sense to include a theme.  Suggestions welcomed.

Hi Nick,
I've been trying to play around with it. I was thinking a picture of the textbook that when clicked on, leads to the chapter sections, making navigation easier. However, I am not sure about copyrighted images so that is just an idea. I've also hyperlinked some words because I found the web addresses to be slightly overwhelming when reading (if that's alright). As for the theme, I am not quite sure what you mean: as in color scheme of the website or general layout?
Thanks!

Can you and Margaret prototype a few options?  It’s fine to include a screenshot of the book cover.
All the best,
Nick

On Jun 6, 2018, at 2:16 PM, sarahmcdonald20<EMAIL_ADDRESS>wrote:
Hi Nick,
I've been trying to play around with it. I was thinking a picture of the textbook that when clicked on, leads to the chapter sections, making navigation easier. However, I am not sure about copyrighted images so that is just an idea. I've also hyperlinked some words because I found the web addresses to be slightly overwhelming when reading (if that's alright). As for the theme, I am not quite sure what you mean: as in color scheme of the website or general layout?
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Done via https://github.com/nicholasjhorton/SDM4inR/commit/2c5e6395fc931b67fd360bb2d4499dc43e605042
Last change for comments by @deveaux69 @pvelleman or @patrick714


Samsung webports updates

Publishing latest fixes related to:

building webports on Windows
using pip3 instead of pip, which may point to pip from python2 package on some Linux distros.

LGTM

LGTM


Support for Visual Studio 2017

On behalf on Johannes Seela
I’m using Visual Studio 2017 (15.0.0+26228.12) and your hello_world.cpp example would not compile. The reason for this is the definition of __BOOST_SML_TEMPLATE_KEYWORD, which is set as
#define __BOOST_SML_TEMPLATE_KEYWORD
#pragma warning(disable : 4503)
#pragma warning(disable : 4200)
#endif
When I change this to
#define __BOOST_SML_TEMPLATE_KEYWORD template
everything works just fine. Seems like VS2017 behaves more like gcc/clang than previous versions (I can’t test this).

Fixed by https://github.com/boost-experimental/sml/commit/ca49165220feb55c5ae1903653b0db02e3c37fb5


fix: fix conditions in the resolveEntity method

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[bug] lastfm cant load after new update.

[2021-04-12 16:13:24] [ERROR] red: Package loading failed
Traceback (most recent call last):
File "/root/redenv/lib/python3.8/site-packages/redbot/core/core_commands.py", line 171, in _load
await bot.load_extension(spec)
File "/root/redenv/lib/python3.8/site-packages/redbot/core/bot.py", line 1355, in load_extension
lib = spec.loader.load_module()
File "", line 462, in _check_name_wrapper
File "", line 962, in load_module
File "", line 787, in load_module
File "", line 265, in _load_module_shim
File "", line 702, in _load
File "", line 671, in _load_unlocked
File "", line 783, in exec_module
File "", line 219, in _call_with_frames_removed
File "/root/.local/share/Red-DiscordBot/data/candy/cogs/CogManager/cogs/lastfm/init.py", line 1, in
from .lastfm import LastFM
File "/root/.local/share/Red-DiscordBot/data/candy/cogs/CogManager/cogs/lastfm/lastfm.py", line 217
async def _api_request(
^
IndentationError: expected an indented block

This will be fixed once #42 is merged.


Create bordered components

Created bordered component on Mission & Values page, will implement in other pages too

great job @scye09 !


Add X-address encoding/decoding RPCs to rippled

I think it would be a good idea to aid adoption of X-address format by adding encoding/decoding RPCs to rippled. Currently sample code is only available for Java and a public API that could potentially be compromised.

The X-address codec is available in all of the supported client libraries:
xrpl.js: https://js.xrpl.org/functions/decodeXAddress.html
xrpl-py: https://xrpl-py.readthedocs.io/en/stable/source/xrpl.core.addresscodec.html
That said, I think it would be convenient to have this as part of rippled. Could you propose an API design for the feature?

Nothing fancy, just like RPC calls of:
encode_x_address(address, optional tag) returns the address in X-format.
decode_x_address(address) returns the norrmal Ripple address and tag.
For safety return an error if you try to decode a testnet address and rippled isn't running on testnet (and vice versa.) When I posted that I don't think X-address format was in any official libraries, you had to go out and get/make external ones.

What prevents a malicious rippled from using an attacker's address in the response?

You'd only want to use that kind of API on a node you run not a public one.


Jump to the newest message

I guess I mentioned it before in some other issue.
I'm getting many, many messages since friendica is combining twitter, diaspora, facebook, pump.io, rss feeds and its own network in one stream. When I haven't used my mobile device for several days I have to scroll up several minutes to be at the current messages. I can't even move by touching the scroll bar, since it is extremely little in that situation.
So it would be great to be able to jump to the newest messages by touching an icon or a menu entry.

We're getting rid of this "Fast Scroll thumb", see #255


New Reputation Type: Botmeter

I don't remember xD @0xjei  I'm going to close it as not planned lol


patch: order

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Handling of connection errors

How can I prevent rethinkdbdash to retry connecting if there were errors or completely get rid of an/the instance?

It automatically tries to reconnect by default.
If you want the driver to get rid of a pool and not try again, use discovery: true


re-enable jasmine tests

Implements first step in #60

Added basic jasmine setup for angular tests
Updated angular.json and package.json to run unit tests

PR looks good to me! Tried it locally and the test runs.
The one action that is failing I think we can ignore. It's just the Slack notification to my own channel to notify me of a PR, but I think there's some security setting involved that disallows PRs from outside forks to use the secret.


Filmstrip view

What's the Problem?
In a folder with many Photos/videos, I would like to preview these files in a larger layout then even the large icon layout provides.
Solution/Idea
I don't remember what it was called but Windows XP's File Explorer used to have what I call a filmstrip view. Where all the files went along the bottom and the majority of the screen was a preview of the photo that was selected. What I liked most about the way Windows XP's File Explorer did it was that only folders that I decided to present this was had this layout. All other folders were in a regular layout.
https://www.techlicious.com/images/computers/windows-xp-filmstrip-view.jpg
Alternatives
Allow for the preview pain to be expanded past 1/5 the screen size it allows now. Currently, the preview pain is Good enough for a quick view. But it could allow for preview to be much larger than it is today.
Priorities
No response
Files Version
<IP_ADDRESS>
Windows Version
Windows 10
Comments
No response

@android272 are you fine with us merging this issue together with #6628?

Yep that is a good solution to this problem.


Hybrid as default Find mode

Set hybrid as the default, and added documentation.

Full long tests -> Gcc Release all failed twice in a row. Will look there when time permits.

It would be nice to cherry-pick 4f498fdf64e02491a39e5a08c476eba9da18d9d1 and 564b41e73973d77598eb5af27d6d6fa402388570 to develop separately.

Please revert all merges except those coming from the develop branch.

No, I'll merge in the PR #286 first, then this PR.

@daniellowell Why are all the changes to the Jenkins file necessary ?

There aren't actually a lot of changes, it is the way git is merging the file. I added an auxilery parallel stage since we are adding a bunch of specialized tests for new sub-features. This is captured in #286

@daniellowell BTW we can make the normal Find the default for DEV builds.


[Question] Support for Varlen Seqs with causal=False

Hi,
I am inquiring about the forward/backward functionality in the FA1.0.7 (or later) and FA2.2 (or later) versions. Specifically, do these versions support varlen sequences when causal=False is set, and the query sequence length (seqlen_q) is not equal to the key sequence length (seqlen_k)?
Thanks!

Yes

Yes

Thanks!


Support OpenDyslexicMono for Broader Accessibility

I understand that requests for fonts are generally ignored, but I do believe this to be a potential exception. I'm also happy to submit a PR for this myself, and hope to confirm it may be considered before getting to work.
Is your feature request related to a problem? Please describe.
Readability is one of the primary goals of carbon, but has natural barriers for the 20% of the population who experience some manner of dyslexia. Dyslexia is an unexpected difficulty in reading for an individual who has the intelligence to be a much better reader. It is most commonly due to a difficulty in phonological processing (the appreciation of the individual sounds of spoken language), which affects the ability of an individual to speak, read, spell and, often, learn a second language.
Describe the solution you'd like
OpenDyslexic is an open-source font family, including a monospaced OpenDyslexicMono. It is distributed under the OFL-1.1 license which allows for modification and distribution and is generally compatible with the MIT license for dependent projects.
https://github.com/antijingoist/opendyslexic
Describe alternatives you've considered
I could potentially host a fork / patch for this feature specifically.

I understand that requests for fonts are generally ignored, but I do believe this to be a potential exception. I'm also happy to submit a PR for this myself, and hope to confirm it may be considered before getting to work.
Is your feature request related to a problem? Please describe. Readability is one of the primary goals of carbon, but has natural barriers for the 20% of the population who experience some manner of dyslexia. Dyslexia is an unexpected difficulty in reading for an individual who has the intelligence to be a much better reader. It is most commonly due to a difficulty in phonological processing (the appreciation of the individual sounds of spoken language), which affects the ability of an individual to speak, read, spell and, often, learn a second language.
Describe the solution you'd like OpenDyslexic is an open-source font family, including a monospaced OpenDyslexicMono. It is distributed under the OFL-1.1 license which allows for modification and distribution and is generally compatible with the MIT license for dependent projects.
https://github.com/antijingoist/opendyslexic
Describe alternatives you've considered I could potentially host a fork / patch for this feature specifically.

@definesfineout I would be open to supporting Open Dyslexic Mono if it can be served from an easily accessible CDN.

@definesfineout any thoughts on implementing this? If you don't have the bandwidth to implement this, I will probably close out this issue as wontfix.

I am interested in implementing this - apologies for the delay, I just had
a major surgery Monday and am recovering.
Re: "easily accessible CDN" any guidelines or criteria there as to what
qualifies as easily accessible? Just so I can be sure I know what I'm
looking for before submitting a PR :)
Thanks!
On Sat, Feb 26, 2022 at 3:34 PM Mike Fix @.***> wrote:

@definesfineout https://github.com/definesfineout any thoughts on
implementing this? If you don't have the bandwidth to implement this, I
will probably close out this issue as wontfix.
—
Reply to this email directly, view it on GitHub
https://github.com/carbon-app/carbon/issues/1332#issuecomment-1052576544,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABWONUMC7RXEBI3WXXDPSDU5E2L3ANCNFSM5NVUU4TA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Defines Fineout
pronouns: any
Engineering Team Lead, Rev.io https://rev.io/
*Secretary of the Board of Directors, Sexual Assault Resource Agency
http://www.saracville.org*540.255.1392 <(540)%20255-1392>

@definesfineout closing this since it is a theme/font issue. But like I mentioned above, please feel free to send a PR whenever you want.


Upgrade Rails to <IP_ADDRESS>

http://weblog.rubyonrails.org/2016/1/25/Rails-5-0-0-beta1-1-4-2-5-1-4-1-14-1-3-2-22-1-and-rails-html-sanitizer-1-0-3-have-been-released/

Default string limit for postgres is not set to 255.  Review monkey-patch in /home/jenkins/workspace/govuk_content-register_branches/config/initializers/remove_default_column_limit.rb.

Thanks @benlovell, we were looking into it.  Fixed now.


operator: Add missing deployment status conditions.

DriverDeployed and CertsVerified in case of verification failure
status conditions were missing.
Also added conditions validation in unit tests.
FIXES #819

Can you also add something to an E2E test? It doesn't need to cover as many cases as the unit tests and it is less likely to break on a real cluster compared to emitting events, but we still need to verify at least the basic path once.

Added the condition validation appropriate tests to cover different cases.

@pohly Do you have any guess what might be causing Testing / 1.17 / works – fedora-1_17.direct-testing-metrics  test failure, which is not supposed to be the result of this change.

Do you have any guess what might be causing Testing / 1.17 / works – fedora-1_17.direct-testing-metrics test failure, which is not supposed to be the result of this change.

There's apparently a small race condition: one metric about CSI calls is only present in the metrics data if at least one CSI call has been recorded. During startup that might not have happened yet. I've add https://github.com/intel/pmem-csi/issues/822


Handle HTML tags

Hi,
First of all thank you very much for this plugin, it is really useful.
I'm running into a problem when using markdown's unordered lists:
-My template:

-Result:

Expected:

I'm not sure what is putting that enter after the title.

Solved by using sanitize. See more in https://github.com/helpers/handlebars-helpers#sanitize
Template:


chore: Move public-api-check script to GH action

The check public-api-script is converted to a GH action with inputs. This is then reused in the AI SDK (PR).
Closes https://github.tools.sap/AI/gen-ai-hub-sdk-js-backlog/issues/115

NOTE
Ignore the many junk commits. I couldn't get nektos/act to run actions locally on my machine, so I had to do some trial and error here.
The public api check works as intended both here and in the AI SDK.
The only problem here is with the unit tests for the public api check. This test case passes in one build/tests action but fails in the other two build/tests actions (see the checks above). According to the current logic, the current code is correct and the test should pass in every case. The issue arises with exportAllInBarrel() not identifying the barrelFile that has been mocked in the test.


proxy: Fix CI fail from format token in Error call

1. What does this change do, exactly?
go vet began failing after 078c991574647c27 and causing CI failures.
This patch changes the Error call added in commit 078c991574647c27 to
an Errorf call to support the use of the %d and %s tokens that are needed.
2. Please link to the relevant issues.
https://travis-ci.org/mholt/caddy/jobs/248392875
https://ci.appveyor.com/project/mholt/caddy/build/2979
caddyhttp/proxy/upstream_test.go:480::error: possible formatting directive in Error call (vet)
3. Which documentation changes (if any) need to be made because of this PR?
None
4. Checklist
This is a ~one~four-character fix, so I'm pretty sure I can safely check these. ;)

[x] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I am willing to help maintain this change if there are issues with it later

 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.

Thank you both, @JoshHarmon and @tw4452852 !


View user profile

Issue reference
Resolves #6
Small description of change

Type of Change

[ ] Fix
[x] Feature
[ ] Documentation Updates
[ ] Breaking Changes

Additional Information

@breta-ionut Move dialog to a new page, use React router parameters user/{id}


No Cities for BC,NB,NF,NT,NS,PE?

Hi @vincentmorneau ,
Seems the dataset omitted everything in half of the provinces.. :-(
C'mon.. Canada, is the centre of the universe, eh?  ;-)

I am in the process of moving to another data source: http://download.geonames.org/export/dump/
Thanks for watching.

Checked against new data source (publishing shortly). Cities are there.

fixed in https://github.com/vincentmorneau/oracle-countries-states-cities/commit/4ef3f9cfcf06d4cd27733da1a66e9ae9dbdc8baf


Suggestion: including changes from nightly Process Hacker

The definitions in this repository are usually somewhat behind those that are available for Process Hacker.
@dmex wrote:

PHNT is supposed to be a stable repository. I test/update types with PH first then sync those changes with phnt so they're more or less guaranteed to work properly.

It is a reasonable approach, but it makes it harder for third-party projects to include the latest changes available for Process Hacker. Currently, if someone wants to use the latest definitions that are not yet available here, they have three options:

Copy necessary header files from Process Hacker.
Fork this repository, merge changes from the Process Hacker's version, and include it as a submodule.
Include the entire Process Hacker as a submodule.

Therefore, I have a suggestion.
How about maintaining both stable and nightly versions of phnt here on different branches? This way, those who need well-tested definitions can use the main branch, while Process Hacker and other projects that want to include the latest types can reference commits on the nightly branch.

Made a standalone version that automatically pulls the phnt folder from master branch if there were any changes to it every night.
If you are interested in using it you can check it out under https://github.com/oberrich/phnt_nightly
Script used to generate the repo: https://gist.github.com/oberrich/d0db45cb71fdbf185ec7c922b6b2b6d6


Multiple entries in VM list show up when cloning fails

Describe the issue
Due to an earlier crash, one of my VMs (call it A.utm) got moved to A 2.utm (but it's still called A in the VM list). When I now try to clone A, I got an error that it couldn't be moved since A 2.utm already exists. After that (I did it again to see if it was a fluke), I now have 2 more VMs called A in the VM list.
Configuration

UTM Version: 4.0.9
macOS Version: 12.5.1
Mac Chip (Intel, M1, ...): M1

I also had this similar thing happen a different way:
I cloned a VM, and after that both have the same name. I then clicked edit on the clone, but didn't change the name. Then, I either clicked cancel or save (can't remember which) and it gave an error that " 2.utm" couldn't be created as it already exists, and then after quitting and reopening the app I had three VMs (seemingly one "ghost" vm from the failed save)

Forgot to close it with https://github.com/utmapp/UTM/commit/05bb6d3546e256a46a7549136023c1040bac1a66

Thanks, will test this with the next release.

Yeah, seems to correctly use the "- 2" name in the UI.


andark: queue redraws and enable fastload

I was using this clock face a lot, so I wanted to improve it. Thanks for the design @HeiLaut
For the draw queuing, I had some problems like:

When unlocked, the second hand would not draw right away.
The second hand would not move on the second, rather it would move a second after being unlocked.
Sometimes locking or unlocking would not trigger the second hand to start drawing.

I tried to address these by using queueDraw and updateState approach from antonclkplus, and it seems to have fixed my problems.
For fast loading, here are my memory leak checks:
 ____                 _
|  __|___ ___ ___ _ _|_|___ ___
|  __|_ -| . |  _| | | |   | . |
|____|___|  _|_| |___|_|_|_|___|
         |_| espruino.com
 2v24 (c) 2024 G.Williams
>Bangle.loadWidgets();Bangle.drawWidgets();
=undefined
>process.memory().usage
=4225
>eval(require("Storage").read("andark.app.js"))
=undefined
>process.memory().usage
=5391
>Bangle.setUI()
=undefined
>process.memory().usage
=4271
>eval(require("Storage").read("andark.app.js"))
=undefined
>process.memory().usage
=5392
>Bangle.setUI()
=undefined
>process.memory().usage
=4271

Alright that's time - thanks for the PR!


Issue #SB-8828 fix: Removed extra scroll bar

Issue https://project-sunbird.atlassian.net/browse/SB-8828

@vinukumar-vs I've tested these changes on 15.6" and 13" device.


Send push notification when someone withdraws from Escrow

Description
Added another poller that monitors Withdrawal events from the Escrow contract and sends a push notification to whoever sent the funds to Escrow (and therefore invited the other user).
Other changes
N/A
Tested
This is deployed on Alfajores. I onboarded a new phone number there after inviting it and received the push notification.
Related issues

Fixes #175

Backwards compatibility
N/A

Hi @gnardini @etuleu Verified task using Android internal build V1.13.0(1004294348 | Test Flight build V1.13.0(55) and found the following:

User  B is not able to get the "Escrow Payment" after confirming phone number if user A has invited with few escrow amount.

Used devices: Google Pixel XL (8.1) , Galaxy M31 (10), Samsung Galaxy A5 (7) , iPhone XR (14).
We already have issue for the same : https://github.com/celo-org/wallet/issues/123
Thanks!!

Hi @gnardini @etuleu Verified task using Android internal build V1.13.0(1004294348 | Test Flight build V1.13.0(55) and found the following:

User  B is not able to get the "Escrow Payment" after confirming phone number if user A has invited with few escrow amount.

Used devices: Google Pixel XL (8.1) , Galaxy M31 (10), Samsung Galaxy A5 (7) , iPhone XR (14).
We already have issue for the same : https://github.com/celo-org/wallet/issues/123


feature: history implementation

hi there,
what would be a good place to start in order to implement history logging for channels?
(writing messages into a DB, keep them for a certain retention period, and display them when when /history is requested by client)
i would like to implement such a feature, what would be a good spot to add such an IO operation?

any feedback / guidance would be highly appreciated :)

I'm working on this in #304 :-)
The backend right now does not use a database, instead an in-memory ring buffer (so retention is limited by count, not time).
Open question: what should the syntax of /history be? grove.io has an example, with an implied wire format, but I'm not sure what other people are doing.

awesome, do you need any help ? :)

You’ve probably seen it, but just in case: ircv3/ircv3-specifications#349

on this, if we do implement full history logging (as in, intended to last longer than just the past 5-50 messages held in memory), we should support multiple storage methods. even if it's just a choice between either flat-file or into a mongo/sqlite db of some kind. full-go would be much preferred, compiling cgo deps isn't too fun :)

and yeye, as mentioned by @jeanm using some standard formats for commands and such would be good... I think to start off, just implementing our own HistoryServ bot (same as NS/CS), along with automatic replay of history if it's enabled in the server config, would make sense. and we can always create the standard commands on top of that.

In terms of full, persistent history logging, my personal preference is for external log bots rather than something integrated into the ircd. (I'm open to integrating persistent history later on but it's not a high priority for me.)
@korovkin yeah, I could be interested in some help later on :-) After #304 gets merged, I have to do some follow-up work (I'm going to add support for automatic history replay on channel join, requested by a specific user). Once that's in place, we would still need implementations of both the draft CHATHISTORY command as specified above, and also probably of a plain HISTORY command (since CHATHISTORY seems quite difficult to use manually). We could either split those, or you could do them both.

#344 was the initial deliverable for this. Further work will be on #348.


Latest nmi_mysql not working due to incorrect import

ImportError: No module named 'utils' at line nmi_mysql.py#L9.
Quick fix is to replace the line with from nmi_mysql.utils import deprecated or from .utils import deprecated

Lol, may issue, di naka reference ung PR.. haha.. reference mo ung PR next time.

#16

Close mo n lng pag natest mo na.

@pprmint okay na. Thanks.


Updated sshrd for new launcher

Updates ios15 ramdisk to use the new launcher created. Works flawlessly on both mac and linux

fix your merge conflicts

fix your merge conflicts

how... the conflicts are literally the changed code...

then drop all the upstream changes and use changes in HEAD

we cant merge this until you fix the merge conflicts


consul integration

The current integration method is to use the public api to notify openduty from an external source like nagios. I'm thinking about the reverse for an external source like consul. If openduty would have access to a local consul agent, it could register a watch for a consul service. The watch events would then be posted into a consul specific openduty api entry point. It will miss however soft/hard state and flapping detection but even it would be nice without them for a first shot.

This is generally a good idea, I'd be really happy to see consul check integration.

@pepov Any update on this ? Or shall i do this? If so, please let me know.

No, we decided to stuck with Nagios in the middle for now, but thanks for asking!


Plunker script source change

All examples shown on website leads to a Plunker page.
The ui-bootstrap javascript code on those pages is 404. So they should be changed from:
From:

To:

Ok fixed.
plunker.zip

@ElitenetDenmark Could you please make a merge request?


Testing that tokens are not visible in pull requests

As discussed with @rogalmic via email

Great, sorry for the useless warning.

I will still look into this.
Thanks to this report i found:
(https://github.com/ChALkeR/notes/blob/master/Stealing-Travis-secure-variables.md)
which is pretty interesting altogether.


Reduce images size

The images take way too long to load ... and rightfully so, one of the image is 16Mo !

I highly recommend using https://squoosh.app/ to optimize image size (massive reductions with no noticeable reduction in quality).
Additionally, if there are images that are still big (over 500K-ish), then maybe addressing via gatsby lazy loading would be an option to pursue.
https://www.gatsbyjs.com/docs/using-gatsby-image/

We can only use Gatsy-imgae for that indeed.
It wasn't made juste due to priority choices. But It shouldn't be super difficult to add


Update copy about stake proportions

How rewards were divided up was not described as currently implemented, so it needs to match what actually happens.

Closed in #34


How to set the input name

I have the demo working but i can't set the input name through the vue component.
<input type="text" placeholder="name" > is what the browser returns.

What is your original vue code ?

this is blade view file.
`
@extends('layouts.app')
@section('content')
	<div class="container">
	BOO
    <autocomplete-vue property="name" :list="[{name: 'test'}, {name: 'test1'}, {name: 'test123'}, {name: 'test2'}, {name: 'test23'}, {name: 'Test'}]" placeholder="name" :auto-hide="true"></autocomplete-vue>
	</div>

@endsection
`

Works as expected. The property attribute just limits the search results to list..


wiki link dont work

What happened?
this link dosnt work
https://github.com/Puyodead1/udemy-downloader/wiki/Downgrade-CDM-to-2209-on-Chrome-(Windows)
Expected Result
work like
https://github.com/Puyodead1/udemy-downloader/wiki/Downgrade-CDM-to-2209-on-Chrome-(Windows)
Branch
master/main
What operating systems are you seeing the problem on?
Windows
Relevant log output
No response
Other information
No response

It works fine


Updating state for sensor.device_trackers took 0.652 seconds.

Logger: homeassistant.helpers.entity
Source: helpers/entity.py:749
First occurred: 16:54:31 (1 occurrences)
Last logged: 16:54:31

Updating state for sensor.device_trackers (<class 'custom_components.spook.ectoplasms.homeassistant.sensor.HomeAssistantSpookSensorEntity'>) took 0.652 seconds. Please report it to the custom integration author.

HA 2023.6 beta, latest spook.
Also see another error that mentions Spook:
Logger: frontend.js.latest.202306011
Source: components/system_log/__init__.py:257
First occurred: 17:01:09 (2 occurrences)
Last logged: 17:01:09

https://t9un4ew5pm1c9q0g5xrz5k7wl67ox9k5.ui.nabu.casa/config/entities?historyBack=1&domain=spook:0:0 ResizeObserver loop completed with undelivered notifications.

Hmm, is this still going on? (Just checking)
../Frenck

Can't say I have, I do have less device_trackers now but I assume others will report when they see the error too.

Seems like something is blocking your event look there. These sensors have no communication or processing at all, so it should be impossible for them to trigger this (unless something blocks the loop).


Do we need both CLOAK_KEYCHAIN and CLOAK_MASTER_KEY or one of them

I'm integrating this extension and it's working great so far! It's simple to set up and it works transparently, I love it! 🙌
I'm curious if we need to provide both CLOACK_KEYCHAIN and CLOAK_MASTER_KEY to our environment when deploying the API.
When using the extension, I see only CLOAK_MASTER_KEY is documented, what to do with CLOAK_KEYCHAIN?
export const prisma = new PrismaClient().$extends(
  fieldEncryptionExtension({
    encryptionKey: process.env['CLOAK_MASTER_KEY'],
  }),
)

Some clarity here would be highly appreciated.

Technically you don't need either.
The keychain system that uses environment variables in 47ng/cloak is not necessary to use prisma-field-encryption, it is actually replicated here when passing multiple decryption keys.
That being said, if you do use the cloak keychain, the master key should not be used for anything else than decrypting the CLOAK_KEYCHAIN environment variable. Your keychain would then contain keys to pass to prisma-field-encryption, along with the fingerprint of the key to use for encryption in the CLOAK_CURRENT_KEY environment variable.
Here's how this could be done (note that there is a lot of data massaging due to the fact that prisma-field-encryption doesn't accept byte arrays for keys, which could be a future improvement I guess):
import { importKeychain, serializeKey } from '@47ng/cloak'
import { fieldEncryptionExtension } from 'prisma-field-encryption'

const keychain = await importKeychain(
  process.env.CLOAK_KEYCHAIN,
  process.env.CLOAK_MASTER_KEY
)

const encryptionKey = await serializeKey(keychain[process.env.CLOAK_CURRENT_KEY])
const decryptionKeys = await Promise.all(Object.values(keychain).map(({ key }) => serializeKey(key)))

export const prisma = new PrismaClient().$extends(
  fieldEncryptionExtension({
    encryptionKey,
    decryptionKeys,
  })
)


Update the README file

#84
The file has been updated by adding some pictures and enhancing it look to make it attractive and drive people to look for the projects

@ombhojane the task has been completed. You can review it. Thank you!

This looks cool!
The model insights image is good, but it's static, right? Please look for a real-time updating way for this or any other platform work!

I have made some changes you can check it @ombhojane. Thank you!

still the images you've mensioned are static, and not proof for validation.
so better is to remove images without base!
also, please have multimodal example usage, if you removed...

Done @ombhojane

Perfect

assign labels here GSSOC Ext 24 and HACKTOBERFEST UNDER THE LEVEL

Perfect

assign labels here GSSOC Ext 24 and HACKTOBERFEST UNDER THE LEVEL

@ombhojane

@ombhojane add "GSSOC-EXT" label on the PR


PubSub blocks Qt widget refreshing

Hi, I am trying to integrate redox pubsub into my Qt GUI application, for publishing financial tick/depth data. I simply connect to both publisher and subscriber.To test redox, I subscribe to a topic with a callback function to handle message received, which simply append the message received to a QPlainTextEdit. And then I publish a message. What happens is that the QPlainTextEdit is not refreshing. Only when I move my mouse or resize the GUI window or click other application window, the message then is displayed in the QPlainTextEdit widget. Looks like by calling publish(), I am blocking the Qt eventloop. Please kindly help. Do I need to use redox pubsub in a seperate Thread? and how? thx

Hi polluxus. Under the hood the redox event loop runs in a separate thread, so I don't see any reason publishing would block the Qt event loop. Can you post your small example program?


Date condition not working

Hi, This code works correctly in Laravel
Post::where('created_at', '>', $date)->get()

but happen error in Alogolia search
Post::search($request->search)->where('created_at', '>', $date)->get()

And error message:
AlgoliaSearch\AlgoliaException with message 'Invalid syntax for numeric condition:created_at=>'

Hello @hamog,
sorry for the very late answer.
The search() method of your model takes as second parameter array of parameters for search. You can see the list of off all parameters in Algolia documentation.
So if you want filter your results according the date, it might look like this:
$result = Post::search('search query', ['filters' => 'date > 1234']);

Be careful as the date has to be numerical. So the best way is to store a timestamp instead of the date.


Implement password reset endpoints

Description:
In order to enhance the security and user-friendliness of our authentication system, we are introducing a crucial feature: the Password Reset functionality. We'll be implementing this feature using the established Passport.js framework and JWT (JSON Web Tokens), ensuring that users can securely reset their passwords when needed, providing a seamless experience within our application.
Tasks:

Integrate the necessary dependencies for Passport.js and JWT within the project.
Configure Passport.js to work seamlessly with the JWT strategy for password reset.
Develop a user registration endpoint, ensuring the creation of new accounts aligns with our stringent security measures.
Create a login endpoint, enabling users to access their accounts securely and generating JWT tokens.
Implement a protected endpoint that mandates a valid JWT token for access.
Set up middleware to authenticate incoming requests using Passport.js and the JWT strategy.
Establish the password reset functionality, allowing users to securely reset their forgotten passwords.
Handle the logout feature to invalidate tokens or sessions, ensuring a clean and secure user experience.

Acceptance Criteria:

Users should be able to register and create new accounts with a focus on security.
Users should be able to log in using their credentials and receive a JWT token for subsequent authenticated requests.
The protected endpoint should grant access only to those with a valid JWT token.
JWT tokens must undergo rigorous validation, rejecting expired or invalid tokens.
Users should have the capability to log out and invalidate their session or token.
The password reset functionality should provide users with a secure and straightforward method to reset their passwords.

Additional Information:

Adhere to industry best practices for security, including robust password hashing and the secure handling of JWT tokens.
Thoroughly document the authentication API endpoints, including essential input parameters and response formats.
Implement comprehensive error handling for various scenarios, such as invalid credentials, expired tokens, or failed password reset attempts.
Maintain a user-friendly experience throughout the password reset process, ensuring user data security.

Related Dependencies or References:

Passport.js: link to documentation
JWT: link to documentation

Can i go with this issue please?

Can i go with this issue please?

Sure

@dilankavishka I will update the description to give you a better idea on what needs to be done.

@anjula-sack Thank you so much

Any updates @dilankavishka?

@anjula-sack i'll finish this as soon as possible.

Any updates @dilankavishka?

almost done with this issue @anjula-sack

When can we get this @dilankavishka?

Can I go with this issue?

Can I go with this issue?

Go ahead @vish2396

@vish2396 hello, any updates on this task ?


add java agent injector

The PR is related to https://github.com/apache/skywalking/issues/8195, and do two things:

Add the java agent injector to replace the original method (injected when building the image) so that we can inject the java agent more natively.
Update the doc.

Do we have a more recommended way to use swck, like via CLI?

Do we have a more recommended way to use swck, like via CLI?

Could you be more specific? CLI(current) is not responsible for deployment AFAIK

Do we have a more recommended way to use swck, like via CLI?

Could you be more specific? CLI(current) is not responsible for deployment AFAIK

I remember there is swctl install to install OAP and UI, it's better we can install or generate agent manifest there too, the current usage in this pr looks kindof complex. For example, user provides their original yaml of the application, and pass them into our command, then our command populates the necessary part like annotations, etc., swctl inject java -f deployments/java-app.yaml | kubectl apply -f -
This is not a blocker of this PR but I hope we can simplify further the usage of our agent injector.

@hanahmily PTAL.

Do we have a more recommended way to use swck, like via CLI?

Could you be more specific? CLI(current) is not responsible for deployment AFAIK

I remember there is swctl install to install OAP and UI, it's better we can install or generate agent manifest there too, the current usage in this pr looks kindof complex. For example, user provides their original yaml of the application, and pass them into our command, then our command populates the necessary part like annotations, etc., swctl inject java -f deployments/java-app.yaml | kubectl apply -f -
This is not a blocker of this PR but I hope we can simplify further the usage of our agent injector.

Maybe we can add it in the future.

@kezhenxu94 We seems don't have any verification in CI process. Should we add some?

@kezhenxu94 We seems don't have any verification in CI process. Should we add some?

I think we don't need functionality testing here, but we should make sure k8s and docker-compose deploying successfully, with at least one metric verified.

@dashanji I think we need a new doc at here, https://skywalking.apache.org/docs/skywalking-java/latest/en/setup/service-agent/java-agent/containerization/#kubernetes
Besides initContainers of a sidecar, SWCK provides another way to deploy agent.

I remember there is swctl install to install OAP and UI, it's better we can install or generate agent manifest there too, the current usage in this pr looks kindof complex. For example, user provides their original yaml of the application, and pass them into our command, then our command populates the necessary part like annotations, etc., swctl inject java -f deployments/java-app.yaml | kubectl apply -f -

The recommended path to onboard java agent injection is to install all SW backends through the SWCK operator. The operator provided several CRDs to manage OAP, UI, and other critical components. This model should be our canonical way.
We could provide a manual-style injection to swctl to help users don't want to inject init container by webhooks automatically. But from experiences of istio, vault, and other projects who have similar injection workflows, the manual way is not widely used, because this style brings a steep learning curve to users since the operator reduced it.

@dashanji I think we need a new doc at here, https://skywalking.apache.org/docs/skywalking-java/latest/en/setup/service-agent/java-agent/containerization/#kubernetes
Besides initContainers of a sidecar, SWCK provides another way to deploy agent.

Yep, I'm writing the blog about how to use SWCK, when it is finished, I will change the doc related to it.

I add an e2e test to ensure that all components can run normally after we build the image.

Updated it.

@dashanji any update?

Any progress to update?

Still testing, I have failed many times on the Github action, I am still looking for the reason.

Could you please re-run the CI job? I feel very strange that the CI running results are different from my personal repository and the test results in my personal repository are good.

Running.

Updated it, thanks for the correction. @kezhenxu94

Hi @dashanji , I updated something and the codes LGTM, can you please check why the tests failed? Once the tests passed we can merge this

Hi @dashanji , I updated something and the codes LGTM, can you please check why the tests failed? Once the tests passed we can merge this

OK

I added a readinessProbe, hope this can make the CI successful.

@kezhenxu94 I run this CI twice, both passed but require 30+ minutes.

Thanks @dashanji for the work!!


Maximum Length of Storage Account is 15 characters

The documentation is largely accurate, however if the Storage Account name is more than 15 characters, we receive this error:
Join-AzStorageAccountForAuth : Parameter -StorageAccountName '[REDACTED]' has more than 15 characters, which is not supported to be used as the SamAccountName to create an Active Directory object for the storage account.   Azure Files will be supporting AES256 encryption for Kerberos tickets, which requires that the SamAccountName match the storage account name. Please consider using a storage account with a shorter name.                                   At line:1 char:1                                                                                + Join-AzStorageAccountForAuth -ResourceGroupName $ResourceGroupName -S ...                                                                                + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                                                                                + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException     + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,Join-AzStorageAccount
If this can be added to the document, it'd likely save a few people needing to go back and recreate accounts with names less than 15 characters.

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

ID: 7a1a9f3d-5733-012a-a0d7-c00ec21b9232
Version Independent ID: 1e70ab95-9488-5aa3-b8ef-e58b3e0067bb
Content: Enable AD DS authentication to Azure file shares
Content Source: articles/storage/files/storage-files-identity-ad-ds-enable.md
Service: storage
Sub-service: files
GitHub Login: @roygara
Microsoft Alias: rogarana

@CodeAsAnInfrastructureGuy Thanks for the question! We are investigating and will update you shortly.

@CodeAsAnInfrastructureGuy Storage account names must be between 3 and 24 characters in length and may contain numbers and lowercase letters only.
Your storage account name must be unique within Azure. No two storage accounts can have the same name.
Can you please share the screen shot of the error message, If the issue still persist, I'd recommend posting your question in https://azure.microsoft.com/en-us/support/community/ We'll follow up there. We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Hi @SumanthMarigowda-MSFT,
The error is in my original post, I don't think an additional screenshot would provide much value.
It looks like if we are using AES256, the Storage Account Name must match the AD SamAccountName of the Computer Object that it creates when running the command. There is a 15 character limitation on Computer Object SamAccountNames it seems.
It looks like this may not be case if not using AES256.
I hope this has been helpful!!

@CodeAsAnInfrastructureGuy Sorry for the delay, I was going back and forth with the product team. Ultimately we've decided not to add anything, since this is an AES restriction. Still, we thank you for bringing this to our attention, and always appreciate feedback. :)
#please-close

I don't agree, I just got this error and it happens even if I set the join script to use RC4. Since AES256 is the limiting factor as @CodeAsAnInfrastructureGuy says, it would save people a lot of time if you just make a comment in the articles that talk about the relevant lengths and restrictions. Since anyone using Azure Files will get this error! We spent a lot of time creating a naming convention and have now found it wont work for Azure files.

@SumanthMarigowda-MSFT-zz @roygara I agree with @dstote at my side we indeed spent some time to figure out this limitation. Would be great to get it added to the main docs.

@SumanthMarigowda-MSFT-zz and @roygara - yes, I'm also agreeing with @CodeAsAnInfrastructureGuy, @dstote, and @cveld.
This is more of an AD DS “SamAccountName” limitation with Kerberos, which is ok for our "Computer Account" AD DS Objects we’ve used and tested etc.
And this should be documented to some fuller degree in storage-files-identity-ad-ds-enable, regardless of it being more to do with Kerberos or AD DS etc.

Also for storage-files-identity-ad-ds-enable, and its check-the-environment section or really for its mention of a set or static SPN created (aka SPN/UPN as "cifs/your-storage-account-name-here.file.core.windows.net"):

I'd also like to ask that there's some mention that this SPN is not configurable later and is really static or expected,
which may be an issue or a consideration later while testing your new Azure Files (SMB) and AD DS Kerberos setup,
and that there may be a need to address the static SPN with a internal/split DNS (with new Zone) for *.file.core.windows.net.

I am posting here to +1 the notion we should be noting the NetBIOS character limit for storage accounts slated for ADDS auth as mentioned above.


can I use several custom converters?

Hi,
I could use both custom converters but not together, but only one or the other:
const conv = json2md.converters
conv.section = function (input, json2md) { return "####-section " + input }
conv.title = function (input, json2md) { return "##-title " + input }
But if I use them together, json2md dies quietly and there is no result at all. How - and is it possible - to use several converters?

It should be possible to use custom converters, of course. Please create a PoC to see how it fails. Thanks!


Updated Python Build Standalone release to 20220630 on 3.10, 3.9, and 3.8.

Addresses #203.

MERGING:

Confirmed that packaged PuPPY works fine on 3.8-3.10 on macOS, Windows, and Linux. :-)

@tmontes looks like 3.8.12 also had an issue in Ubuntu 22.04 with X11, as seg faults when loading a PyQt5 app (more info in https://github.com/mu-editor/mu/issues/2318).
This has been merged, but it's unreleased, could we make a new release to include this? I think this is the only inclusion, so should be fairly safe:
https://github.com/mu-editor/pup/compare/1.0.0a17...master


Firebase Cloud messaging

2020.3601. notifications.registerForPushNotifications({useFCM=true})
it does not trigger the confirmation message to authorise the sending notifications.
And didn’t receive Firebase Message.
The same works fine in Android. Thanks

@Shchvova any update?

@Shchvova

This is old but I was able to register my device on my iPhone 11 and get notifications with the newest Firebase Notification v2 plugin