Envoy 1.7.1 compile

Hi, I am having some issues compiling this using the provided guide in the README.md with 1.7.1.
Starting local Bazel server and connecting to it...
DEBUG: /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/io_bazel_rules_go/proto/def.bzl:138:3: You no longer need to call proto_register_toolchains(), it does nothing
External dependency cache directory /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f
make: Entering directory '/home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps'
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/cares.dep.env
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/libevent.dep.env
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/gperftools.dep.env
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/yaml-cpp.dep.env
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/benchmark.dep.env
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/luajit.dep.env
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/nghttp2.dep.env
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/zlib.dep.env
make: Leaving directory '/home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps'

real	0m0.045s
user	0m0.015s
sys	0m0.011s
DEBUG: /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy/bazel/repositories.bzl:76:5: Fetching external dependencies...
DEBUG: /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy/bazel/repositories.bzl:82:5:
DEBUG: /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy/bazel/repositories.bzl:83:5: External dependency cache directory /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f
make: Entering directory '/home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps'
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/cares.dep.env
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/libevent.dep.env
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/gperftools.dep.env
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/yaml-cpp.dep.env
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/benchmark.dep.env
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/luajit.dep.env
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/nghttp2.dep.env
No need to rebuild /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps_cache_2c744dffd279d7e9e0910ce594eb4f4f/zlib.dep.env
make: Leaving directory '/home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy_deps'

real	0m0.045s
user	0m0.015s
sys	0m0.011s
DEBUG: /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/envoy/bazel/repositories.bzl:84:5: External dep build exited with return code: 0
WARNING: /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/com_github_grpc_grpc/BUILD:1972:1: in srcs attribute of cc_library rule @com_github_grpc_grpc//:grpc_nanopb: please do not import '@com_github_grpc_grpc//third_party/nanopb:pb_common.c' directly. You should either move the file to this package or depend on an appropriate rule there. Since this rule was created by the macro 'grpc_generate_one_off_targets', the error might have been caused by the macro implementation in /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/com_github_grpc_grpc/bazel/grpc_build_system.bzl:172:12
WARNING: /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/com_github_grpc_grpc/BUILD:1972:1: in srcs attribute of cc_library rule @com_github_grpc_grpc//:grpc_nanopb: please do not import '@com_github_grpc_grpc//third_party/nanopb:pb_decode.c' directly. You should either move the file to this package or depend on an appropriate rule there. Since this rule was created by the macro 'grpc_generate_one_off_targets', the error might have been caused by the macro implementation in /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/com_github_grpc_grpc/bazel/grpc_build_system.bzl:172:12
WARNING: /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/com_github_grpc_grpc/BUILD:1972:1: in srcs attribute of cc_library rule @com_github_grpc_grpc//:grpc_nanopb: please do not import '@com_github_grpc_grpc//third_party/nanopb:pb_encode.c' directly. You should either move the file to this package or depend on an appropriate rule there. Since this rule was created by the macro 'grpc_generate_one_off_targets', the error might have been caused by the macro implementation in /home/master/.cache/bazel/_bazel_master/dade022b46ca8a8ec61a5f966e5c06df/external/com_github_grpc_grpc/bazel/grpc_build_system.bzl:172:12
INFO: Analysed target //http-filter-modsecurity:envoy (290 packages loaded).
INFO: Found 1 target...
ERROR: /home/master/ModSecurity-envoy/http-filter-modsecurity/BUILD:13:1: Linking of rule '//http-filter-modsecurity:envoy' failed (Exit 1)
/usr/bin/ld.gold: error: http-filter-modsecurity/libmodsecurity.a:1:9: syntax error, unexpected STRING
/usr/bin/ld.gold: error: http-filter-modsecurity/libmodsecurity.a: not an object or archive
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilterConfig::HttpModSecurityFilterConfig(modsecurity::Decoder const&): error: undefined reference to 'modsecurity::ModSecurity::ModSecurity()'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilterConfig::HttpModSecurityFilterConfig(modsecurity::Decoder const&): error: undefined reference to 'modsecurity::ModSecurity::setConnectorInformation(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >)'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilterConfig::HttpModSecurityFilterConfig(modsecurity::Decoder const&): error: undefined reference to 'modsecurity::ModSecurity::setServerLogCb(void (*)(void*, void const*), int)'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilterConfig::HttpModSecurityFilterConfig(modsecurity::Decoder const&): error: undefined reference to 'modsecurity::Rules::loadFromUri(char const*)'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilter::HttpModSecurityFilter(std::shared_ptr<Envoy::Http::HttpModSecurityFilterConfig>): error: undefined reference to 'modsecurity::Transaction::Transaction(modsecurity::ModSecurity*, modsecurity::Rules*, void*)'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilter::~HttpModSecurityFilter(): error: undefined reference to 'modsecurity::Transaction::~Transaction()'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilterConfig::~HttpModSecurityFilterConfig(): error: undefined reference to 'modsecurity::ModSecurity::~ModSecurity()'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilter::onDestroy(): error: undefined reference to 'modsecurity::Transaction::processLogging()'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilter::decodeHeaders(Envoy::Http::HeaderMap&, bool)::{lambda(Envoy::Http::HeaderEntry const&, void*)#1}::operator()(Envoy::Http::HeaderEntry const&, void*) const: error: undefined reference to 'modsecurity::Transaction::addRequestHeader(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilter::decodeHeaders(Envoy::Http::HeaderMap&, bool): error: undefined reference to 'modsecurity::Transaction::processURI(char const*, char const*, char const*)'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilter::decodeHeaders(Envoy::Http::HeaderMap&, bool): error: undefined reference to 'modsecurity::Transaction::processRequestHeaders()'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilter::decodeData(Envoy::Buffer::Instance&, bool): error: undefined reference to 'modsecurity::Transaction::appendRequestBody(unsigned char const*, unsigned long)'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilter::decodeData(Envoy::Buffer::Instance&, bool): error: undefined reference to 'modsecurity::Transaction::processRequestBody()'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilter::encodeHeaders(Envoy::Http::HeaderMap&, bool)::{lambda(Envoy::Http::HeaderEntry const&, void*)#1}::operator()(Envoy::Http::HeaderEntry const&, void*) const: error: undefined reference to 'modsecurity::Transaction::addResponseHeader(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilter::encodeHeaders(Envoy::Http::HeaderMap&, bool): error: undefined reference to 'modsecurity::Transaction::processResponseHeaders(int, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilter::encodeData(Envoy::Buffer::Instance&, bool): error: undefined reference to 'modsecurity::Transaction::appendResponseBody(unsigned char const*, unsigned long)'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function Envoy::Http::HttpModSecurityFilter::encodeData(Envoy::Buffer::Instance&, bool): error: undefined reference to 'modsecurity::Transaction::processResponseBody()'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function modsecurity::debug_log::DebugLog::DebugLog(): error: undefined reference to 'vtable for modsecurity::debug_log::DebugLog'
/usr/bin/ld.gold: the vtable symbol may be undefined because the class is missing its key function
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function modsecurity::RulesProperties::RulesProperties(modsecurity::debug_log::DebugLog*): error: undefined reference to 'modsecurity::audit_log::AuditLog::AuditLog()'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function modsecurity::RulesProperties::RulesProperties(modsecurity::debug_log::DebugLog*): error: undefined reference to 'modsecurity::RulesExceptions::RulesExceptions()'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function modsecurity::RulesProperties::RulesProperties(modsecurity::debug_log::DebugLog*): error: undefined reference to 'modsecurity::RulesExceptions::~RulesExceptions()'
bazel-out/k8-fastbuild/bin/http-filter-modsecurity/_objs/http_filter_lib/http_filter.pic.o:http_filter.cc:function modsecurity::RuleMessage::log[abi:cxx11](modsecurity::RuleMessage const*, int): error: undefined reference to 'modsecurity::RuleMessage::log[abi:cxx11](modsecurity::RuleMessage const*, int, int)'
collect2: error: ld returned 1 exit status
Target //http-filter-modsecurity:envoy failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 11.234s, Critical Path: 4.27s
INFO: 0 processes.
FAILED: Build did NOT complete successfully

Facing same issue..@ekarlso were you able to resolve this?

Envoy just released 1.10.0. 1.7.1 is very old and the there have been many deprecations and changes since then. It looks like this filter was thrown over the wall at a point in time and not maintained since.

I've updated Envoy to 1.11.0


Several python imports not found

Hey,
I just installed python3 with homebrew on my mac, together with the pip module.
I installed the requirements.txt using python3 -m pip install -r requirements.txt, which worked fine.
Now when trying to run the main.py file using the python launcher 3 for macos, it shows me the following missing import errors.

Is there anything wrong with the way I tried to run your script ?
Yours Sincerely,
Interface

Additional Info, It only shows the errors for all mt2.(Module) Modules.

I can suggest a temporary fix which would be remove mt2 from the import and have it as utils.amf

I tried to do it with every mt2 import which couldn't be found, still giving errors because It can't find tlsclient-x86.dylib.
I assume I will have to wait until you push a fix for that.

Yes the tlsclient hasn’t been setup to be dynamic with other platforms yet. Although the code is referenced  in the client src as it’s still very much new

Works now, after fixing the imports.

Its giving a warning still:
/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/urllib3/connectionpool.py:1099: InsecureRequestWarning: Unverified HTTPS request is being made to host 'disco.mspapis.com'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#tls-warnings
warnings.warn(
And as soon as I try any option within your script, it will instantly say : "press any button to get back to home" without doing the given action.

That error is fine for now, and only buy_clothes work no other option has been implemented to public yet as it’s a alpha version port of the csharp msptool

All good, if you need any more help just let me know. I am also a dev, we are currently working on a KeyAuth replacement. Mainly focusing on security.

Join discord.gg/hasurei


[Android][Column rendering] [TextBlock inside column inside another column not rendering]

Copied bug from: https://github.com/microsoft/AdaptiveCards/issues/5415
Details from original post:
Platform
What platform is your issue or question related to? (Delete other platforms).

[ ] Android

Author or host
Microsoft Teams
Version of SDK
SDK version 2.4.0
Details
Text Block inside column inside another column does not render.
Sample JSON - Link
Before

After

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

ID: 3fb4394d-25b1-8752-3996-fe3a2934732d
Version Independent ID: 6d391381-89d0-117c-27cc-00043bd26a36
Content: Cards reference - Teams
Content Source: msteams-platform/task-modules-and-cards/cards/cards-reference.md
Product: msteams
GitHub Login: @surbhigupta12
Microsoft Alias: surbhigupta12

@dipja  - We are looking into this I will get back to you soon.

@dipja - Could you please check the alignment (Width , height)  properly in you json file.
Example:-"items":[
{
"columns":[
{
"width":"120px",
"items":[
{
"height":"32px",
"style":"person",
"type":"Image",
"url":"https://urlp.sfbassets.com/urlpcdn/sa1/contentapi1/f16ddd44-5869-47c0-8dd0-81bd8bc578ce",
"width":"32px"
}
],
"type":"Column"
},

@dipja- Could you please confirm if your issue has resolved with above suggestion or still looking for any help?

We are closing this issue for now, Please feel free to reopen it, if required.


Issues deploying lately

Hi there, I was looking at deploying this following the guidance in #10 however I run into an error when starting sbt.
[ec2-user@ip-10-0-0-28 XSL-tester]$ sbt
[info] Loading project definition from /home/ec2-user/XSL-tester/project
[error] Server access Error: Connection refused (Connection refused) url=http://repo.typesafe.com/typesafe/ivy-releases/com.typesafe.play/sbt-plugin/scala_2.10/sbt_0.13/2.2.4/ivys/ivy.xml
[error] Server access Error: Connection refused (Connection refused) url=http://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.typesafe.play/sbt-plugin/scala_2.10/sbt_0.13/2.2.4/ivys/ivy.xml
[error] SERVER ERROR: HTTPS Required url=http://repo1.maven.org/maven2/com/typesafe/play/sbt-plugin_2.10_0.13/2.2.4/sbt-plugin-2.2.4.pom
[error] Server access Error: Connection refused (Connection refused) url=http://repo.typesafe.com/typesafe/releases/com/typesafe/play/sbt-plugin_2.10_0.13/2.2.4/sbt-plugin-2.2.4.pom
[warn] 	module not found: com.typesafe.play#sbt-plugin;2.2.4

Wondering if anyone has tried deploying this recently, does the project need to have updated sbt or scala versions?

I was able to work around this by defining a repositories file and pointing sbt to it.
contents of /home/ec2-user/.sbt/repositories
  local
  sbt-releases-repo: https://repo.typesafe.com/typesafe/ivy-releases/, [organization]/[module]/(scala_[scalaVersion]/)(sbt_[sbtVersion]/)[revision]/[type]s/[artifact](-[classifier]).[ext]
  sbt-plugins-repo: https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/, [organization]/[module]/(scala_[scalaVersion]/)(sbt_[sbtVersion]/)[revision]/[type]s/[artifact](-[classifier]).[ext]
  maven-central: https://repo1.maven.org/maven2/
  typesafe-releases: https://repo.typesafe.com/typesafe/releases
  typesafe-ivy-releases: https://repo.typesafe.com/typesafe/ivy-releases, [organization]/[module]/(scala_[scalaVersion]/)(sbt_[sbtVersion]/)[revision]/[type]s/[artifact](-[classifier]).[ext]

and execute sbt with a flag to point to this repositories file and override the cache (not sure if the second flag is always required).
sbt -Dsbt.repository.config=/home/ec2-user/.sbt/repositories -Dsbt.override.build.repos=true
But there's probably a better way of handling it, I guess depending on the sbt environment maybe?


Ability to add markers to held maps

As worlds grow it becomes increasingly more difficult to remember important locations or points of interest even with the usage of maps. The idea would be to allow dyes to be applied to maps at the current player position so color codes can be used to indicate whatever the player desires. Could use soap or some other item to act as an eraser and remove markers within a certain radius of the player's position.
I would use purple for example to indicate nether portal locations so I can better visualize my network.

Definitely something that could be valuable to add. I'll be honest that something like this would be lower priority to add, but is definitely something I'd be willing to look into at some point.

i love this idea and support it with my heart ^_^

I have posted a first attempt https://github.com/BTW-Community/BTWMapMarkers. Feel free to absorb into the BTW CE and modify as needed. Thanks for the encouragement!


.aac support

Easy to implement for audio, not easy to get metadata

Done, now I have to fix #103


The current fetching data (/tagKeys) is logged with Exception while fetching data (/tagKeys) : null

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
OAP server (apache/skywalking)
What happened

What you expected to happen
Exception while fetching data (/tagKeys) : null
How to reproduce
Exception while fetching data (/tagKeys) : null
Anything else
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Check your logs, we don't respond questions like this.


Pop up an error when deleting one Azure AD attached ADLS Gen2 blob container

Storage Explorer Version: 1.10.1
Build: 20191024.4
Branch: master
Platform/OS: Windows 10/ Linux Ubuntu 19.04/macOS High Sierra
Architecture: ia32/x64
Regression From: Previous release(1.10.1)
Steps to reproduce:

Give RBAC access to an ADLS Gen2 blob container to a user like 'Blob Data Owner Role'.
Attach this ADLS Gen2 blob container via Azure Active Directory (Azure AD).
Try to delete the Azure AD attached blob container.
Check the result.

Expect Experience:
No error occurs and delete successfully.
Actual Experience:
Pop up an error.

@v-limu Make sure you connect with a DFS endpoint. We've added code to make sure you connect with that endpoint. I'm unable to reproduce with a DFS endpoint.

We're missing interop delete for ADLS Gen2.


implement suggestions from joss reviewer

leading underscore convention for private attributes
delete unused type attributes
untrack notebook not used in documentation
small style improvements
<3.12 constraint to python version due to numba compatibility

Thanks for the input here.
This doesn't quite address our workflow with the documentation, so I'd like
to request that we move on from this and leave it be.
Thanks,
Chris
On Fri, Oct 20, 2023 at 4:46 AM raphaelreinauer @.***>
wrote:

@.**** commented on this pull request.
On docs/notebooks/bullseye.ipynb
https://github.com/scikit-tda/DREiMac/pull/13#discussion_r1366672190:
This might be helpful:
https://mg.readthedocs.io/git-jupyter.html
—
Reply to this email directly, view it on GitHub
https://github.com/scikit-tda/DREiMac/pull/13#discussion_r1366672190,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJWDZW2JM5ZNUCS6RC3ZTLYAI24JAVCNFSM6AAAAAA6DTZHKGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMOBZGU2DSMBXGE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


ESM only monorepo w/project references error without main fallback

Bug Report

🔎 Search Terms
yarn workspaces
typescript project references
main vs exports
ESM only

🕗 Version & Regression Information
4.7.2 this is new and experimental according to the handbook
⏯ Playground Link
This is a full hello world repo example on a branch, simple and straightforward minimal reproduction using git reset (also described in the readme):

git clone --single-branch --branch bug-main-required-to-build https://github.com/rosskevin/ts-esm-workspaces.git
yarn install && yarn build this will error with TS2307: Cannot find module
git reset --hard HEAD~1 && yarn build will succeed.  The difference is adding back the main field to the package.json

💻 Code
As described by the  handbook regarding ESM:
  "exports": {
    ".": {
      "types": "./dist/index.d.ts",
      "import": "./dist/index.js"
    }
  },

should work.
🙁 Actual behavior
In our attempt to convert to ESM only (not hybrid and dropping support for CJS), it fails with:
~/p/ts-esm-workspaces ❯❯❯ yarn clean && yarn build
packages/client/src/presentSomething.ts:1:36 - error TS2307: Cannot find module '@rosskevin/ts-esm-workspaces-shared' or its corresponding type declarations.

1 import { inc, appendMessage } from "@rosskevin/ts-esm-workspaces-shared";

🙂 Expected behavior
In the handbook it says:

Node.js supports a new field for defining entry points in package.json called "exports". This field is a more powerful alternative to defining "main" in package.json, and can control what parts of your package are exposed to consumers.

alternative is the keyword here...this OR that.  In reality, a build will only succeed with a main, which is a BAD signal indicating that CJS is available (and it is not).

As an aside, I hope to thread through all issues in this repo and hope it may stand as a hello-world monorepo/project references/yarn workspaces example for ESM.  I do hope the effort to put this together helps others and helps iron out onboarding issues for ESM only libraries. @see https://github.com/rosskevin/ts-esm-workspaces.git

@rosskevin I was only able to get exports resolution working by setting "moduleResolution": "nodenext".

Thanks @milesj, my concern with that was the ambiguity of targets.  Considering 1. shared is agnostic; 2. cli is node; and 3. client is browser targeted, I was thinking that I needed standard node resolution.  I would like some clarity on this specifically for this situation.

I think I have a similar issue. Since a recent upgrade on another branch in my monorepo, I get the errors
Cannot find type definition file for 'node'

in the tsconfig of all packages in the repo and
Cannot find module

in each of the .ts/.tsx files in the applications.
I've tried downgrading to another version but it seems that other packages have TS referenced with "*" as a dependency. The only way of having a working branch at the moment is reverting to an old commit and never run yarn install...
Is there another solution?

My only question at this point is if there is a consequence to using nodenext resolution for a browser environment?

Only that browsers don't do any kind of node-like resolution at all and so none of your imports are guaranteed to work at runtime. That was true with node module resolution, too, though. In reality, you're probably using some bundler that uses node's resolution rules if it was OK before, and hopefully said bundler supports node's newer resolution features, too.

There is now an FAQ Entry explaining that to respect export maps we require moduleResolution be node16 or nodenext.

This is an FYI to those that are trying to migrate larger applications to ESM.
While I could get a project references + yarn workspaces hello  world working, in real life, it failed.  Partially because projects like @apollo/client have not adopted the proper exports specification.  Using "moduleResolution": "nodenext" results in:

Here is the reproduction: https://github.com/rosskevin/ts-esm-workspaces/tree/apollo-exports
This is not the only dependency that fails in this manner, and it seems somehow correlated to the use of export * from.
Whereas "moduleResolution": "node" succeeds.  IMPORTANTLY - it only takes one incorrect dependency to constrain you from using nodenext resolution.
This is a HUGE adoption problem.  Whereas the typescript project may be implementing the spec properly, if ANY of my dependencies do not, I am out of luck.  This ultimately means: I have to switch back to "moduleResolution": "node" AND I have to FAKE the main in the project.json so that typescript will build without error.
I don't know the right answer to this dilemma, but it certainly puts users between a rock and a hard place, ensuring that we don't do things properly and prolonging this period between legacy and ESM.

@rosskevin Yup this is a huge problem, I even tweeted about how many projects are either a) not using exports correctly or b) not shipping types correctly. I've worked around some of them by using yarn patches, while others are straight blockers. I foresee the ecosystem taking a while to fix all of these issues.

A direct descendant to this issue has been filed, please take the conversation there: https://github.com/microsoft/TypeScript/issues/49388


OS.get_environment doesn't work on Android

Tested versions
No matter which environment variable I want to query on Android, the result is empty.
System information
Android 14, Godot 4.2
Issue description
No matter which environment variable I want to query on Android, the result is empty.
Steps to reproduce
Versuche einfach eine Environment Variable unter Android auszugeben
Minimal reproduction project (MRP)
func _ready() -> void:
label.text = "DCIM: " + OS.get_environment("ANDROID_HOME")

No matter which environment variable I want to query on Android, the result is empty.

Which environment variables have you tried? ANDROID_HOME isn't defined by default on Android to my knowledge.
Note that Android has a lot of security measures in place, so I wouldn't be surprised if this was blocked on non-rooted devices.

try "DIRECTORY_DCIM", should be the directory for the captured images. An empty string is returned.

CC @m4gr3d

Works as expected, just doesn't support a lot of env variables.
For example it works with the PATH env variable: print("FHK - Testing env variable: " + OS.get_environment("PATH")) returns:
FHK - Testing env variable: /product/bin:/apex/com.android.runtime/bin:/apex/com.android.art/bin:/system_ext/bin:/system/bin:/system/xbin:/odm/bin:/vendor/bin:/vendor/xbin

From a quick test on a Pixel 5 device, here are the supported env variables (retrieved using System.getenv()):
13:22:40.018 MainActivity             D  FHK - Printing Env variables:
13:22:40.018 MainActivity             D  FHK - SYSTEMSERVERCLASSPATH
13:22:40.019 MainActivity             D  FHK - PATH
13:22:40.019 MainActivity             D  FHK - ANDROID_SOCKET_zygote
13:22:40.019 MainActivity             D  FHK - ANDROID_I18N_ROOT
13:22:40.019 MainActivity             D  FHK - FIRST_STAGE_SNAPUSERD_PID
13:22:40.019 MainActivity             D  FHK - ANDROID_DATA
13:22:40.019 MainActivity             D  FHK - ASEC_MOUNTPOINT
13:22:40.019 MainActivity             D  FHK - ANDROID_TZDATA_ROOT
13:22:40.019 MainActivity             D  FHK - EXTERNAL_STORAGE
13:22:40.019 MainActivity             D  FHK - ANDROID_BOOTLOGO
13:22:40.019 MainActivity             D  FHK - ANDROID_ASSETS
13:22:40.019 MainActivity             D  FHK - FIRST_STAGE_SNAPUSERD_INFO
13:22:40.019 MainActivity             D  FHK - TMPDIR
13:22:40.019 MainActivity             D  FHK - ANDROID_STORAGE
13:22:40.019 MainActivity             D  FHK - STANDALONE_SYSTEMSERVER_JARS
13:22:40.019 MainActivity             D  FHK - DEX2OATBOOTCLASSPATH
13:22:40.019 MainActivity             D  FHK - ANDROID_ART_ROOT
13:22:40.019 MainActivity             D  FHK - ANDROID_ROOT
13:22:40.019 MainActivity             D  FHK - DOWNLOAD_CACHE
13:22:40.019 MainActivity             D  FHK - BOOTCLASSPATH
13:22:40.019 MainActivity             D  FHK - ANDROID_SOCKET_usap_pool_primary


[Question] How to CRUD comments with Graphql interface?

Thought there're few query schema generated automatically,
I didn't see any Mutation schema in there
Should I customized the schema.graphql by myself? or It's just a bug or feature missing?

Hi @ash0080 we didn't provide support for GQL yet. I will be putted to the roadmap soon. Happy if you can contribute to it :)

Here is how I extended the plugin in my project if it can help:
module.exports = {
  definition: `
    extend type CommentsComment {
      children: [CommentsComment]
    }
  `,
  query: `
    commentsInHierarchy(relation: String): [CommentsComment]
  `,
  resolver: {
    Query: {
      commentsInHierarchy: {
        description: 'Return the comments in hierarchy for a relation',
        resolverOf: 'plugins::comments.comments.findAllInHierarchy',
        resolver: async (obj, options, ctx) => {
          return await strapi.plugins.comments.controllers.comments.findAllInHierarchy({ params: { relation: options.relation }});
        },
      },
    },
  },
};

I only did it for one method but it can easily be extended

Would potentially be willing to sponsor, if interested. My e-mail is<EMAIL_ADDRESS>Thank you

Would potentially be willing to sponsor, if interested. My e-mail is<EMAIL_ADDRESS>Thank you

I'd like to see graphql support. It will be great if you implement it!

Here is how I extended the plugin in my project if it can help:
module.exports = {
  definition: `
    extend type CommentsComment {
      children: [CommentsComment]
    }
  `,
  query: `
    commentsInHierarchy(relation: String): [CommentsComment]
  `,
  resolver: {
    Query: {
      commentsInHierarchy: {
        description: 'Return the comments in hierarchy for a relation',
        resolverOf: 'plugins::comments.comments.findAllInHierarchy',
        resolver: async (obj, options, ctx) => {
          return await strapi.plugins.comments.controllers.comments.findAllInHierarchy({ params: { relation: options.relation }});
        },
      },
    },
  },
};

I only did it for one method but it can easily be extended

What file do you add this to?

We'd like too see GraphQL support too.


Option to refresh file defined in using statement in Bicep Parameters file

Is your feature request related to a problem? Please describe.
When you are developing a schema (User defined type) for template often you make some changes to the file references in the using statement in Bicep parameters file. Those changes are not refreshed when using intellisense. Workaround is to change the reference file to some other and than change back. Give us some VSC command that we can use to refresh or better VSC to check for changes in that file and automatically refreshes it.
Describe the solution you'd like

@slavizh could you give more detail?
In general, this should work - for example if I rename a param or change its type in the .bicep file, the .bicepparam file immediately goes red. The logic to detect edits is the same as for module references.
The gaps I'm currently aware of with detections to edits:

If the .bicep file is edited outside of VSCode.
Changes to files referenced via the load* functions (e.g. loadJsonContent).

@anthony-c-martin I have tried different scenarios and does not work for me. Mainly things like:

add new property to user defined type
changed description for user defined types
added new param

none of these work for me.
Keep that I am editing both .bicep and .bicepparam files in VSC but in different windows as the .bicepparam is not in the same folder as the .bicep file.

I spoke too early. on the adding new param and new property. Seems those are reflected although with some delay. I have noticed that they are now visible after writing the above reply.

so I think it definitely does not work for description of user define types.

so I think it definitely does not work for description of user define types.

@slavizh would you mind providing a step-by-step repro? I've tried to repro this myself but have been unable.

ok.
The simplest way to reproduce it was:

Open VSC window. Create file main.bicep in some folder. Do not add anything for now
Open another VSC windows. Create config.bicepparam file in another folder with using statement pointing to the main.bicep. My value was something like using '../../../../folder1/main.bicep'
In main.bicep add code like this:

type fooObj = {
  @description('My description 1')
  name: string
  @description('My Description 2')
  value: string
}

param test fooObj

Go back to config.bicepparam and try to see if intellisense will list test param. In my case it was not listed. After I have closed main.bicep in VSC the property appeared.

you can also try changing the descriptions of those properties and see if they are updated. In my case they are not updated even when I close main.bicep after edit.
Both of these files are in different git repositories and while editing them I have not committed the changes. Not sure if that has some relationship to the issue.

Thanks for the detailed repro!
This is unfortunately a known limitation - VSCode provides us with notifications for file changes within a workspace, but if you've got 2 editor windows open, they're treated as 2 different workspaces, and so we don't get the file change notifications.
It looks like to support this behavior fully, we'd basically have to DIY it ourselves and use the FileSystemWatcher class to do our own watching outside of VSCode.
I'll update the issue title to make this clearer.

@anthony-c-martin this raises another question. If I am referring to module in container registry how do you know if the module was updated or not? The particular example is if I have bicepparam file with using statement towards container registry. As we know in order to achieve scenario where people fetch the latest schema from module without having to change the using statement all the time we usually publish a few tags. So for version 1.2.3 we would publish tags 1, 1.2 and 1.2.3. That way users can refer to 1 just to get the latest schema with latest changes and features available. So let's say tag 1 is updated. What happens with the bicepparam file that is referencing it. Will it detect that the reference is updated or not? If not how people should get the latest version? My wish is that it automatically detects the newer version without having the end user to do anything or execute some commands.

That way users can refer to 1 just to get the latest schema with latest changes and features available. So let's say tag 1 is updated. What happens with the bicepparam file that is referencing it. Will it detect that the reference is updated or not? If not how people should get the latest version? My wish is that it automatically detects the newer version without having the end user to do anything or execute some commands.

My understanding is that there's no logic to detect changes to a tag (once it has been cached locally, the cache won't be evicted), and that generally updating an existing tag is best avoided. I'm not an expert with the registry logic - @shenglol & @majastrz should be able to give more detail.

@shenglol thanks. the command works as intended.

Dup of https://github.com/Azure/bicep/issues/8132.
@slavizh are you okay with closing this and https://github.com/Azure/bicep/issues/8132 as by design?  Thx.

@StephenWeatherford yep


broken link to supporte forum listing page

this link puts me in an endless loop back to the subscriber msdn page.
I'm trying to get to the listing of supported forums shown in this page
https://docs.microsoft.com/en-us/visualstudio/subscriptions/vs-priority-support
for several weeks now.  please let me know how to find it again

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

ID: 50d40a75-d2eb-44ab-656f-4ea80109c54d
Version Independent ID: a069527e-6c7f-7d51-4aa6-82ad8be32bf1
Content: Priority Support Benefit in Visual Studio Subscription
Content Source: subscriptions/vs-priority-support.md
Product: vs-subscription
GitHub Login: @evanwindom
Microsoft Alias: jaunger

@gpadvorac Thanks for your feedback! We'll investigate this and report back.
@evanwindom @lankaim @prrana Is this something we can clarify/fix in the documentation?

@gpadvorac
@TerryGLee
The link in the tile has been redirected to a forum list on the MSDN Support Team Blog as an interim solution.  Thank you for the report.


Middleware Pipe marked as final

In order to efficiently use dependency injection, I need to be able to use multiple MiddlewarePipes; specifically one for the core middleware pipeline, and one for the route specific middleware pipeline.
With a final class this is a real challenge as my DI can return a new instance each time or a shared instance, depending on how I configure it. Neither is workable.
If I were able to extend the class with CoreMiddleware and RouteMiddleware it would be fine, and work as expected.
What was the reasoning to make the class final, as in v2 it worked as expected?!?

With a final class this is a real challenge as my DI can return a new instance each time or a shared instance, depending on how I configure it. Neither is workable.

Please depend on the MiddlewarePipeInterface:
https://github.com/zendframework/zend-stratigility/blob/9ea8435eac09be8ec2f6eafa72f5d41f11e2e823/src/MiddlewarePipeInterface.php#L15-L18
As for final, the complete and generic explanation about the adoption of more and more of the final keyword can be found at https://ocramius.github.io/blog/when-to-declare-classes-final/
The correct approach is to use a decorator that receives a MiddlewarePipeInterface as constructor argument (along with whatever you need), and implements MiddlewarePipeInterface itself, delegating execution when appropriate.

The decision was made because (a) the implementation is quite trivial, (b) we're providing MiddlewarePipeInterface now, which means (c) you can achieve things you'd normally do via extension using composition or by implementing the new interface.
I'm not sure what DI container you're using, exactly, so it's hard to provide examples. Perhaps you can give some more details? In zend-servicemanager, you could provide a factory that creates and returns your MiddlewarePipe instance, allowing you to map services to specific factories. Alternately, implement the interface, which allows you to have discrete classes.


Incorrect parameter name in README.MD

Hi, in the readme it says that the the parameter for the server url is 'url'. It actually appears to be host.

Nice catch, cheers. I've changed the option to server since host is a little misleading. host will also still work for the foreseeable future.


Debug panel doesn't work with pretty Url

What steps will reproduce the problem?

Use public $enablePrettyUrl = true; in UrlManager
Enable yii2-debug panel in config
When you visit the page using pretty URL - debug panel doesn't appear. Seems like this is the reason:

When you visit the page without pretty URL - debug panel is OK.

What is the expected result?
Debug panel works fine with pretty URL.
What do you get instead?
Debug panel doesn't work fine.
Additional info
Yii2 advanced template.

Q
A

Yii version
2.0.13

PHP version
5.6

Operating system

I'm using it w/ pretty URLs daily. Something else should be the reason.

@samdark Hello. Thank you for the answer. What info can I give you to define the reason?

The first pic shows a redirect response.
What do your url rules look like?

@Alex-Code
yes, redirect
Url rules about /debug/ ?

The difference on the screenshot is trailing /.

Do you have URL normalizer turned on?

@samdark yes. and I have $suffix = '\';

Aha, that's something to check.

Aha, that is the reason.

@samdark I'm sorry, I see that you created bug issue for next Yii2 milestone, but can you please advice me what to do? Any manipulations with UrlNormalizer or $suffix don't work for me. Thanks.

That's for next debug toolbar release. Framework itself is doing fine.
If you want it bad you can dig into the problem, fix it in debug toolbar and send a pull request. We'll review/merge it and then you'll be able to set your debug dependency to dev-master@dev until release is tagged.

@samdark Hello. I noticed next thing - when I'm removing my $suffix = '/'; from UrlManager - debug works fine, but I need this slash. Can you please advice me what should I write special rule for debug in UrlManager to disable slash only for debug?
'rules' => [ 		// ... 		'debug/<controller>/<action>' => 'debug/<controller>/<action>', 	],
That feature doesn't work for me. My thanks


Added ability to select a region for cloud server

Created dropdown to select region from list of 5 cities (static list for now)
Passed the user's region selection through uproxy_core_api.CloudInstallArgs to the cloud provider
Still needs UI work.

Great, thanks so much for taking this on.

So close! I love how this is looking. Just a couple of code things to iron out.

:+1:, yay!

Just saw the latest screens, good stuff!
On Wed, Mar 9, 2016 at 5:29 PM, Laura<EMAIL_ADDRESS>wrote:

Merged #2299 https://github.com/uProxy/uproxy/pull/2299.
—
Reply to this email directly or view it on GitHub
https://github.com/uProxy/uproxy/pull/2299#event-584034025.

Yay!


fix: articles/app-service-mobile/app-service-mobile-html-how-to-use-client-library.md

Add code fence language and fix comment format

@nschonni : Thanks for your contribution! The author, @conceptdev, has been notified to review your proposed change.

It might also be 2 separate blocks with the comments outside, but I went for the smaller change

#sign-off


#70 - implemented new AbstractFactory for PsrLoggerAdapter

Add feature described in https://github.com/zendframework/zend-log/issues/70

[X] Are you creating a new feature?

[ ] Add documentation for the new feature.
[X] Add a CHANGELOG.md entry for the new feature.

[X] Is this related to documentation?

Documentation is missing. Any suggestion where to place it exactly. There is a part ServiceManager and one for PSR-3. In which of them could we put it in?

@dennybrandes Put the documentation under the ServiceManager section, but reference and link to it from the PSR-3 section.

@weierophinney I added documentation now.

@dennybrandes I've incorporated the feedback from @froschdesign and pushed back to your branch. Thanks for your contribution!


fix: search component improvement

fix: 'recentItem' code moved to a component
fix: eslint update -- return default forbid elements and instead checkContextTypes was set to true
global styled added for shared styles
eslint update -- import/prefer-default-export -- updated to remove lint error.
styled component with single export was throwing a lint error

@nexxtway/rainbow-algolia-search

I moved the components for the SerachInput and RecentItem to components folder.
This might be the last push to this PR


Redesign of AUI:Selector and AUI:Editor

A total eclipse of the heart redesign of the above interfaces.

@animaux was this supposed to look like the below? Seems somewhat buggy/incomplete to me.

This only works in conjunction with the updated UI Selector.

Will have to rebase this, since there are too many changes in the master.


Create process for running unit tests

Hello,
I have wrote a tutorial to setup karma with your sample library. I am not sure that is the best way, but it works.
You may be inspired to automate the process.
See ng2-testable-lib
Thank you for your project.

@raphael-volt — Thank you for sharing.
I have added a link to your tutorial in the README:
https://github.com/jvandemo/generator-angular2-library#how-can-i-configure-karma
Thanks again, much appreciated! 👍


making the entire row clickable

I want to add a custom code on a row click (selection is only on selection)
Isn't there a more elegant way to do it other than overriding rowTemplate (by copy pasting it from the code and add ng-click)?
It doesn't sound really forward compatible (ui-grid-row.html might change in the future)

You could use directive stacking (like this project does) to add additional functionality to the row directive.
Have a look at how the features do this.


Support for Newsweek videos?

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version
$ LC_ALL=C.UTF-8 \youtube-dl -U
It looks like you installed youtube-dl with a package manager, pip, setup.py or a tarball. Please use that to update.

[X] I've verified and I assure that I'm running youtube-dl 2019.02.08

Before submitting an issue make sure you have:

[X] At least skimmed through the README, most notably the FAQ and BUGS sections
[X] Searched the bugtracker for similar issues including closed ones
[X] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[X] Feature request (request for a new functionality)
[ ] Question
[ ] Other

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
$ \youtube-dl -vvv https://www.newsweek.com/time-reversed-quantum-computer-1361215
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-vvv', u'https://www.newsweek.com/time-reversed-quantum-computer-1361215']
WARNING: Assuming --restrict-filenames since file system encoding cannot encode all characters. Set the LC_ALL environment variable to fix this.
[debug] Encodings: locale ANSI_X3.4-1968, fs ANSI_X3.4-1968, out ANSI_X3.4-1968, pref ANSI_X3.4-1968
[debug] youtube-dl version 2019.02.08
[debug] Python version 2.7.15+ (CPython) - Linux-4.18.0-16-generic-x86_64-with-Ubuntu-18.10-cosmic
[debug] exe versions: ffmpeg 4.0.2-2, ffprobe 4.0.2-2, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[generic] time-reversed-quantum-computer-1361215: Requesting header
WARNING: Falling back on generic information extractor.
[generic] time-reversed-quantum-computer-1361215: Downloading webpage
[generic] time-reversed-quantum-computer-1361215: Extracting information
ERROR: Unsupported URL: https://www.newsweek.com/time-reversed-quantum-computer-1361215
Traceback (most recent call last):
  File "/home/kenorb/.local/lib/python2.7/site-packages/youtube_dl/extractor/generic.py", line 2337, in _real_extract
    doc = compat_etree_fromstring(webpage.encode('utf-8'))
  File "/home/kenorb/.local/lib/python2.7/site-packages/youtube_dl/compat.py", line 2542, in compat_etree_fromstring
    doc = _XML(text, parser=etree.XMLParser(target=_TreeBuilder(element_factory=_element_factory)))
  File "/home/kenorb/.local/lib/python2.7/site-packages/youtube_dl/compat.py", line 2531, in _XML
    parser.feed(text)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1659, in feed
    self._raiseerror(v)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1523, in _raiseerror
    raise err
ParseError: not well-formed (invalid token): line 68, column 74
Traceback (most recent call last):
  File "/home/kenorb/.local/lib/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 794, in extract_info
    ie_result = ie.extract(url)
  File "/home/kenorb/.local/lib/python2.7/site-packages/youtube_dl/extractor/common.py", line 508, in extract
    ie_result = self._real_extract(url)
  File "/home/kenorb/.local/lib/python2.7/site-packages/youtube_dl/extractor/generic.py", line 3320, in _real_extract
    raise UnsupportedError(url)
UnsupportedError: Unsupported URL: https://www.newsweek.com/time-reversed-quantum-computer-1361215

$ \youtube-dl https://www.newsweek.com/6f51dea4-43e9-467c-9bf2-bfddec0478bb
WARNING: Assuming --restrict-filenames since file system encoding cannot encode all characters. Set the LC_ALL environment variable to fix this.
[generic] 6f51dea4-43e9-467c-9bf2-bfddec0478bb: Requesting header
WARNING: Could not send HEAD request to https://www.newsweek.com/6f51dea4-43e9-467c-9bf2-bfddec0478bb: HTTP Error 410: Gone
[generic] 6f51dea4-43e9-467c-9bf2-bfddec0478bb: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 410: Gone (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.newsweek.com/time-reversed-quantum-computer-1361215
Copied video address: blob:https://www.newsweek.com/24ea3be9-e4fe-46ca-9432-065510410acd

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
I was expecting that videos from Newsweek would be supported, but they're not. I believe they're using similar mechanism to the other sites.

Newsweek downloads chunks of the video as it plays, the blob URL itself is unused.
In your case, when you play the video it requests https://video.newsweek.com/transcoder/480hls/2550/time-reversal-16x9-cms-155250170000000.ts
Which replies as 206, and then it iterates the last number on<PHONE_NUMBER>00000.ts till the full video is loaded. Ie<PHONE_NUMBER>00001.ts -><PHONE_NUMBER>00002.ts -><PHONE_NUMBER>00003.ts
These just return MPEG-2 chunks.
I'm surprised that youtube-dl doesn't natively handle this?


@ #208 | should increase the amount of inotify watchers on Ubuntu

@phuonglm please review

need sudo sysctl -p after update sysctl.conf

@phuonglm PR updated

PR ok for me.


Switch option for automatic break

Summary of the new feature / enhancement
Consider this switch statement:
switch ($x)
{
    1 {"It is one."; Break}
    2 {"It is two." ; Break }
    3 {"It is three." ; Break }
    4 {"It is four." ; Break }
}

It would be much more readable if we could:
switch ($x) -AutoBreak
{
    1 {"It is one."}
    2 {"It is two." }
    3 {"It is three."}
    4 {"It is four."}
}

Proposed technical implementation details (optional)
No response

In the first one the break is not needed.
Break is only there for the case of two non-exclusive options and where you don't want switch to process other members of an array,
 > $x = 5,4,3
> switch ($x)
>> {
>>     1 {"It is one."; Break}
>>     2 {"It is two." ; Break }
>>     3 {"It is three." ; Break }
>>     4 {"It is four." ; Break }
>> }
It is four.

Doesn't break (or do anything) for the first item, breaks on the second, skipping the third.  Where additional items need to be processed continue is needed so  there would need to be an -autocontinue option as well as the break.
I do see people who put continue / break in every block (including the the default one which doesn't run if anything else did)  they're "goto" commands and should be used very sparingly, but with regex switches, sometimes it is easier to use them than to code mutually exclusive regexes.
However changing an command so there is a version which works in new versions and fails in old versions is usually more trouble than its worth.

I like the idea, as the need for explicit break or continue` (with array-valued input, in case further elements should be processed) is not intuitively obvious, cumbersome and easy to forget in practice.
To me, it is a classic case of having gotten the default behavior wrong.
However, the desirable default behavior is actually continue, not break: with scalar input, it acts the same as break; with array-value (collection) input, it moves on to the next element, whereas break instantly exits the switch statement.
That is, your switch - which should arguably therefore be called -AutoContinue - should behave as follows (make all of the continue statements below unnecessary):
# Scalar input
# -> 1 instead of @(1, 2)
switch -Regex ('foo') {
  'o' { 1; continue }  # without `continue`, the 'f' branch would be evaluated too
  'f' { 2; continue }
}

# Array-valued input
# -> @(1, 3) instead of @(1, 2, 3, 4)
switch -Regex ('foo', 'bar') {
  'o' { 1; continue }
  'f' { 2; continue }
  'a' { 3; continue }
  'b' { 4; continue }
}

Your example doesn't explain why this is needed. With or without break, the result is the same.

The result is not the same (maybe in this example). Break will exist on first thing found. With larger switches and particularly with expressions, break may be needed for performance reasons. Having to repeat it all the time is just noise, and to have an option is easily maintainable. Furthermore, there is no compatibility problem.

@mklement0

That is, your switch ... should arguably therefore be called -AutoContinue

But this is not the same as far as I understand it. Perhaps there is merit to have both options or one with the parameter : -Auto break|continue

@majkinetor, my example was meant to demonstrate that it is the same: continue behaves like break with a single input value, and with multiple input values I assume that you want to continue processing the remaining input values, right?
In other words: We want implicit per-input-value break behavior, which is what continue does.
If you really want to short-circuit processing further values, you can always use break, but my assumption was that this is not the desired default behavior.

@mklement0

and with multiple input values I assume that you want to continue processing the remaining input values, right?

I am not sure if you can assume that :) Clearly, the code is NOT the same with multiple input values. While you may be right that it is indeed what people would expect, one can expect that opposite phenomena exists in the wild.

Well, it is the default behavior in the absence of continue and break.
Fair enough: I wouldn't complain about having both -AutoBreak and -AutoContinue (which implies that in the single-input-value case the two would amount to the same behavior).

break may be needed for performance reasons

Switch with 1e6 switches? pwsh will rather crash. For example, 1001 generated switches, look at duration:

& {
  $x = 0..1000
  $exp =
  @"
      switch (`$x)
      {
          $(foreach ($i in $x)
          {
              if ($i -eq 999) { "{0} {{{0}; {1}}}`n" -f $i, 'break' }
              else { "{0} {{`$null}}`n" -f $i }
      })}
  "@

      iex $exp
  }

@237dmitry, while the number of branches will indeed be limited, the number of input values needn't be - and there, performance matters.
Consider the runtime of the following, both with and without break; even use of continue - which still processes all input values - is an important optimization with many input values.
switch (1..1e6) {
  { $_ % 2 -eq 1 } { <# ... #> break }
  { $_ % 2 -eq 0 } { <# ... #> }
}


Changes in the battery plugin and in the Atomic Theme

Improvements in the theme.
Added pmset command that improves the process of detecting the battery and its percentage in OSX.

Thanks! I did some timing on this and it looks like pmset is slightly faster than the previous ioreg command. Nice find!
❯ time ioreg -n AppleSmartBattery -r | awk '$1~/Capacity/{c[$1]=$3} END{OFMT="%05.2f%%"; max=c["\"MaxCapacity\""]; print (max>0? 100*c["\"CurrentCapacity\""]/max: "?")}'
96.16%

real	0m0.035s
user	0m0.007s
sys	0m0.030s

❯ time pmset -g ps | sed -n 's/.*[[:blank:]]+*\(.*%\).*/\1/p'
100%

real	0m0.022s
user	0m0.010s
sys	0m0.011s


[kqueue] hits unreachable code with high completion loads

I totally might be using the library incorrectly. I am just starting out working with it and trying to set it up to function as an http server. Right now I just have the basics hardcoded. If I benchmark it with wrk and a high number of connections (512) I end up seeing: error(libxev_kqueue): invalid state in submission queue state=backend.kqueue.Completion.State.active and an eventual crash with the following stacktrace:
thread 7376789 panic: reached unreachable code
/opt/homebrew/Cellar/zig/0.13.0/lib/zig/std/debug.zig:412:14: 0x100ff6c63 in assert (platform)
    if (!ok) unreachable; // assertion failure
             ^
/Users/bren077s/.cache/zig/p/1220612bc023c21d75234882ec9a8c6a1cbd9d642da3dfb899297f14bb5bd7b6cd78/src/queue.zig:24:19: 0x100ff515f in push (platform)
            assert(v.next == null);
                  ^
/Users/bren077s/.cache/zig/p/1220612bc023c21d75234882ec9a8c6a1cbd9d642da3dfb899297f14bb5bd7b6cd78/src/backend/kqueue.zig:225:38: 0x100ff2763 in submit (platform)
                self.completions.push(c);
                                     ^
/Users/bren077s/.cache/zig/p/1220612bc023c21d75234882ec9a8c6a1cbd9d642da3dfb899297f14bb5bd7b6cd78/src/backend/kqueue.zig:333:24: 0x100ff6dd7 in tick (platform)
        try self.submit();
                       ^
/Users/bren077s/.cache/zig/p/1220612bc023c21d75234882ec9a8c6a1cbd9d642da3dfb899297f14bb5bd7b6cd78/src/backend/kqueue.zig:264:62: 0x100ff8b47 in run (platform)
            .until_done => while (!self.done()) try self.tick(1),
                                                             ^
/Users/bren077s/Projects/roc-coro/platform/src/main.zig:63:22: 0x100ff8fcb in main (platform)
    try root_loop.run(.until_done);
                     ^
/opt/homebrew/Cellar/zig/0.13.0/lib/zig/std/start.zig:524:37: 0x100ff96af in main (platform)
            const result = root.main() catch |err| {
                                    ^
???:?:?: 0x18e714273 in ??? (???)
???:?:?: 0x0 in ??? (???)
run
└─ run platform failure
error: the following command terminated unexpectedly:
/Users/bren077s/Projects/roc-coro/platform/zig-out/bin/platform

I don't think this is the same as #111, but I might be wrong.
Any general advise would be greatly appreciated (even if the advice is that libxev is not ready and I should really switch to something else). I'm not trying to build anything production ready, just working on a prototype of a coroutine based webserver to use with roc (which is a new programming language).
The full repo at the current state is here: https://github.com/bhansconnect/roc-coro-webserver/tree/9b07612a95d1b937b0b8e7ec90a14c1a310f5b2b/platform
Here is a slightly reduced version of the main file that is leading to the errors:
const std = @import("std");
const xev = @import("xev");
const log = std.log.scoped(.platform);
const Allocator = std.mem.Allocator;

pub const std_options: std.Options = .{
    .log_level = .info,
};

var gpa: std.heap.GeneralPurposeAllocator(.{}) = .{};
var allocator: Allocator = gpa.allocator();

pub fn main() !void {
    var root_loop = try xev.Loop.init(.{});
    defer root_loop.deinit();

    var address = try std.net.Address.parseIp4("<IP_ADDRESS>", 8000);
    const server = try xev.TCP.init(address);

    // Bind and listen
    try server.bind(address);
    try server.listen(128);

    // Is this needed? I think it is getting the port.
    // But we specify a specific port...
    const fd = if (xev.backend == .iocp) @as(std.os.windows.ws2_32.SOCKET, @ptrCast(server.fd)) else server.fd;
    var sock_len = address.getOsSockLen();
    try std.posix.getsockname(fd, &address.any, &sock_len);
    log.info("Starting server at: http://{}", .{address});

    // Setup accepting connections
    var c_accept: xev.Completion = undefined;
    server.accept(&root_loop, &c_accept, void, null, (struct {
        fn callback(
            _: ?*void,
            loop: *xev.Loop,
            _: *xev.Completion,
            accept_result: xev.AcceptError!xev.TCP,
        ) xev.CallbackAction {
            const socket = accept_result catch |err| {
                log.err("Failed to accept connection: {}", .{err});
                return .rearm;
            };
            log.debug("Accepting new TCP connection", .{});
            var handler = allocator.create(Handler) catch unreachable;
            socket.read(loop, &handler.completion, .{ .slice = &handler.buffer }, Handler, handler, Handler.read_callback);
            return .rearm;
        }
    }).callback);

    try root_loop.run(.until_done);
}

const Handler = struct {
    const Self = @This();

    completion: xev.Completion,
    buffer: [4096]u8,

    fn read_callback(
        self: ?*Self,
        loop: *xev.Loop,
        _: *xev.Completion,
        socket: xev.TCP,
        rb: xev.ReadBuffer,
        len_result: xev.ReadError!usize,
    ) xev.CallbackAction {
        const len = len_result catch |err| {
            // I'm not sure this is correct, but I think we need to retry on would block.
            // Feels like something that libxev should handle on its own.
            if (err == error.WouldBlock) {
                return .rearm;
            }
            if (err != error.ConnectionResetByPeer and err != error.EOF) {
                log.warn("Failed to read from tcp connection: {}", .{err});
            }
            self.?.close(loop, socket);
            return .disarm;
        };
        // TODO: handle partial reads.
        // I'm a bit suprised that reading 0 bytes doesn't count as would block.
        if (len == 0) {
            return .rearm;
        }
        log.debug("Request: \n{s}\n", .{rb.slice[0..len]});

        // TODO: This is where we should parse the header, make sure it is valid.
        // Check the full lengh and keep polling if more is to come.
        // Also should check keep alive.

        const response =
            "HTTP/1.1 200 OK\r\nContent-Type: text/plain; charset=utf-8\r\nContent-Length: 13\r\n\r\nHello, World!";
        const slice = std.fmt.bufPrint(&self.?.buffer, response, .{}) catch |err| {
            log.warn("Failed to write to io buffer: {}", .{err});
            self.?.close(loop, socket);
            return .disarm;
        };
        socket.write(loop, &self.?.completion, .{ .slice = slice }, Self, self, Self.write_callback);
        return .disarm;
    }

    fn write_callback(
        self: ?*Self,
        loop: *xev.Loop,
        _: *xev.Completion,
        socket: xev.TCP,
        wb: xev.WriteBuffer,
        len_result: xev.WriteError!usize,
    ) xev.CallbackAction {
        const len = len_result catch |err| {
            // I'm not sure this is correct, but I think we need to retry on would block.
            // Feels like something that libxev should handle on its own.
            if (err == error.WouldBlock) {
                return .rearm;
            }
            if (err != error.ConnectionResetByPeer and err != error.EOF) {
                log.warn("Failed to write to tcp connection: {}", .{err});
            }
            self.?.close(loop, socket);
            return .disarm;
        };
        // TODO: handle partial writes.
        // I'm a bit suprised that writing 0 bytes doesn't count as would block.
        if (len == 0) {
            return .rearm;
        }
        log.debug("Response: \n{s}\n", .{wb.slice[0..len]});

        // Send back to reading. Just assuming keep alive for now.
        socket.read(loop, &self.?.completion, .{ .slice = &self.?.buffer }, Self, self, Self.read_callback);
        return .disarm;
    }

    fn close(self: *Self, loop: *xev.Loop, socket: xev.TCP) void {
        socket.close(loop, &self.completion, Self, self, close_callback);
    }

    fn close_callback(
        self: ?*Self,
        _: *xev.Loop,
        _: *xev.Completion,
        _: xev.TCP,
        _: xev.ShutdownError!void,
    ) xev.CallbackAction {
        // If shutdowns fails, should this retry?
        allocator.destroy(self.?);
        return .disarm;
    }
};

Thanks for creating libxev and any help. Initial perf tests of this libxev are looking really good.

As an extra note. Did a quick test on a linux machine. Everything seems to work without hiccup on linux.

Thanks for the report! I will check this out soon.

One other added note, I realized that this is probably the same as: #105

Ok, I'm not sure the best solution yet, but I think I found the core issue. If loop.run(.no_wait) is used, the kqueue backend with hit this break:  https://github.com/mitchellh/libxev/blob/b8d1d93e5c899b27abbaa7df23b496c3e6a178c7/src/backend/kqueue.zig#L540-L541
This break can happen when there are still changes left from here: https://github.com/mitchellh/libxev/blob/b8d1d93e5c899b27abbaa7df23b496c3e6a178c7/src/backend/kqueue.zig#L523-L527
If there are any changes left, they are never passed to kqueue to clear things up. So those extra changes either need to be delayed to the next call to loop.run(), or they need to be cleaned up when breaking out of the loop here.

Note, a minimal changes that works around the bug (but obviously leads to waiting) is:
if (wait == 0 and changes == 0) break;

Can you try https://github.com/mitchellh/libxev/pull/115 ?

No, #115 is not enough to fix my issue above. I have #115 along with a few other kqueue changes here and still haven't fully fixed the issues I am hitting with kqueue and high concurrency: https://github.com/mitchellh/libxev/compare/main...bhansconnect:libxev:main#diff-caa3f8691afca2b16eaf30614ff2616aa0d6cb38bbe2750f39e7669e4902ff81


Fake review detection

A machine learning model which can predict whether an online review is fraudulent or not. The main idea used to detect the fake nature of reviews is that the review should be computer generated through unfair means. If the review is created manually, then it is considered legal and original.
Detection of fake reviews out of a massive collection of reviews having various distinct categories with each review having a corresponding rating, label i.e. CG(Computer Generated Review) and OR(Original Review generated by humans) and the review text.
Main task is to detect whether a given review is fraudulent or not. If it is computer generated, it is considered fake otherwise not.
Techniques used:
Removing punctuation character
Transforming text to lower case
Eliminating stopwords
Stemming
Lemmatizing
Removing digits
Transformers Used for Text Vectorization, Weighting and Normalization:
CountVectorizer Bag of Words Transformer
TFIDF(Term Frequency-Inverse Document Frequency) Transformer
I would like to work on these issue

Could you assign this to me? I'm interested.  I have hands-on Python and Machine Learning.

I requested to assign me the issue. @sanjay-kv

Let me know @sreesti07 if you want to collaborate with @Satyam0775

@Satyam0775  I will collabrate with @sreesti07 ,

Yea I am okay to collaborate with @Satyam0775

I am excited to tackle this issue. I requested you to assign me

Yea I am okay to collaborate with @Satyam0775

okey

Hello Sir i am also want to contribute in this issue .


feat: update anatomy icons

Update anatomy icons' parts

@anubra266 The website doesn’t build, can you look into that?

It's fixed now @segunadebayo


[Consideration] Principle of adopting analysis tools

Originally suggested by @mgjeong (https://github.com/lf-edge/edge-home-orchestration-go/pull/193#issuecomment-742469991)
We need to establish the principle in adopting analysis tools for security vulnerability, code quality, and so on, which is consent from all our community developers. One good example is LGTM, which is currently adopted as validating security vulnerabilities in our project.
Once we complete establishing the principle, then we will make a decision to keep our current adopted tools or to take an alternative (including LGTM).

In my opinion, good code analysis systems will help us improve the quality of our product. Before adding any system, we can raise the issue and discuss it. I think a good reference point would be the collaboration/use of the practices described at OpenSSF.
I also suggest analyzing the checklist taken from this organization:
The following checks are all run against the target project:

Name
Description

Security-MD
Does the project contain a security policy?

Contributors
Does the project have contributors from at least two different organizations?

Frozen-Deps
Does the project declare and freeze dependencies?

Signed-Releases
Does the project cryptographically sign releases?

Signed-Tags
Does the project cryptographically sign release tags?

CI-Tests
Does the project run tests in CI, e.g. GitHub Actions, Prow?

Code-Review
Does the project require code review before code is merged?

CII-Best-Practices
Does the project have a CII Best Practices Badge?

Pull-Requests
Does the project use Pull Requests for all code changes?

Fuzzing
Does the project use OSS-Fuzz?

SAST
Does the project use static code analysis tools, e.g. CodeQL, SonarCloud?

Active
Did the project get any commits and releases in last 90 days?

Regarding the supported analysis systems in our project:
LGTM (@MoonkiHong added) system has already shown good results and it helps us make more competitive code.
CodeQL (@t25kim added) helps track quality during the pull request stage.
@MoonkiHong @t25kim thank you very much for your contribution for improving our project.

@tdrozdovsky Thank you for the valuable idea.

In my opinion, good code analysis systems will help us improve the quality of our product. Before adding any system, we can raise the issue and discuss it.

I fully agree with your idea of discussing some tools usability after opening an issue.
I think it would be nice to apply the tool during the PoC(?) period and see if it should be adopted by checking the feasibility.

The following is the result of scorecard at OpenSSF.
RESULTS
-------
Active: Pass 10
CI-Tests: Pass 10
CII-Best-Practices: Pass 10
Code-Review: Pass 10
Contributors: Pass 10
Frozen-Deps: Fail 5
Fuzzing: Fail 10
Pull-Requests: Pass 9
SAST: Fail 9
Security-Policy: Fail 10
Signed-Releases: Fail 10
Signed-Tags: Fail 0

@tdrozdovsky Thank you for the valuable idea.

In my opinion, good code analysis systems will help us improve the quality of our product. Before adding any system, we can raise the issue and discuss it.

I fully agree with your idea of discussing some tools usability after opening an issue.
I think it would be nice to apply the tool during the PoC(?) period and see if it should be adopted by checking the feasibility.

@t25kim Fully agree with you. Thank you for your suggestion!!!

The following is the result of scorecard at OpenSSF.
RESULTS
-------
Active: Pass 10
CI-Tests: Pass 10
CII-Best-Practices: Pass 10
Code-Review: Pass 10
Contributors: Pass 10
Frozen-Deps: Fail 5
Fuzzing: Fail 10
Pull-Requests: Pass 9
SAST: Fail 9
Security-Policy: Fail 10
Signed-Releases: Fail 10
Signed-Tags: Fail 0

@t25kim This is fantastic! Thank you for your proactive analysis through scorecard from OpenSSF.

The checklist that I offered above is just overview of the implementation of the ScoreCard, an example of the results of which was provided by the @t25kim (thank you). Therefore, I understand that you (@MoonkiHong, @t25kim) support the direction of ScoreCard use in our project. Now let's wait for the opinion of other development participants.
P.S. I hope that if other maintainers support this idea, we will implement it step by step together.
Thank you again


Revert the changes of the upgrate to keycloak 3.2.0.Final

I created this PR in case we want to revert the changes, we did to move to 3.2.0.Final.

Cool.
Note: the deployment of 3.2.0.Final anyway didn't happen. yet.


Word repeating issue

ADD ISSUE DESCRIPTION HERE
Upon typed word, the editor instead of leaving the word, it duplicates it. Also seen more of a sentence gets duplicated
Also active on MEDION TABLET ANDROID 10, running extra keyboard: Microsoft SwiftKey
Version: 1.74.2
Commit: e8a3071ea4344d9d48ef8a4df2c097372b0c5161
User Agent: Mozilla/5.0 (Linux; Android 10; Lenovo TB-X306F) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: github.dev

duplicate


Update scancode processing

Fixes #621
Fixes #355
This is a significant re-write of xrdp keyboard handling.
See also xorgxrdp PR neutrinolabs/xorgxrdp#316
This builds on the work in #3039 / #3113 with significant changes:-

The ad-hoc ways of indexing scancodes have been brought together in a 'scancode index' which maps an RDP scancode into an 8-bit value. This index can then be used to access an array containing state information about a particular key.
RDP scancode defines are provided in a new development file xrdp_scancode_defs.h
(related to the above) A lot of magic numbers have been removed.
Some keys which only worked with xorgxrdp (pause and ABNT2 keypad point) now work with VNC as well.
The confusingly named function xrdp_load_keyboard_layout() which is only needed for xorgxrdp has been renamed xrdp_init_xkb_layout() and is now only called when needed.
The keycode set (base or evdev) can be specified for the X server at run-time.The corresponding XKB rules are passed to xorgxrdp.
A minor unreported problem with xorgxrdp has been resolved.

I've got a bit more testing to do before I can merge this, but I'd welcome any comments.

Unless anyone's got any great objections, I'll merge this and neutrinolabs/xorgxrdp#316 into devel after we've shipped v0.10.1 (see #3155). It seems pretty solid to me, but I've not got a huge array of keyboards here for testing. Getting it out into the community should flush out any last few problems before we ship it in a major release.


Running game in editor breaks system audio

Godot version:

3.0.6.stable.custom_build

OS/device including version:

Linux 4.20.11-arch2-1-ARCH
pulseaudio 12.2
nvidia 418.43-2
01:00.0 VGA compatible controller: NVIDIA Corporation GM204 [GeForce GTX 970] (rev a1)
01:00.1 Audio device: NVIDIA Corporation GM204 High Definition Audio Controller (rev a1)

Issue description:

When I start godot and run my game before attempting to play any other
audio/video on my system, I am no longer able to play audio/video through other
programs.  Spotify says "Can't play the current song", youtube videos hang,
ect.
If I play a video/song before playing my game, I can continue doing so even
after running my game.  This only occurs when I run my game in godot before
trying to play audio/video through another program.  It isn't just opening
godot that causes this, but specifically running a scene in godot.
This might just be some oddity I need to figure out about my system, but I thought I'd post here in case others have a similar experience.
Steps to reproduce:

Open a godot project
Run your game
Open spotify/youtube/ect.
Try to play a song/video
Video won't play, spotify says "Can't play the current song"

Steps that won't reproduce:

Open spotify/youtube/ect.
Try to play a song/video
Open a godot project
Run your game

or

Open a godot project
Open spotify/youtube/ect.
Try to play a song/video
Run your game

Could you test with the latest 3.1 beta build?

@rcorre Are you sure you're actually using PulseAudio in your system? Sound to me like you're using ALSA in which audio devices can't be shared by the applications.
Can you run godot with the --verbose flag? That should show what audio driver is actually being used on the console.

@marcelofg55 looks like you're correct!
[rcorre@midova ~]$ godot --verbose
Audio Driver ALSA average latency: 21ms (frame=940)

Turns the AUR has a separate godot-pulse package, the plain godot package uses pulseaudio=no.

Yup, that was it. Thanks!


Take dataframe conversion out of read_jsonlines

Yes, that's right. I'm going to add a property-based test in another PR.


fix: don't support callback as first arg when params are required in node

https://github.com/twilio/twilio-node/pull/876

@claudiachua No worries. Even I approved them.


Codecov Report

Merging #14758 (0ccdc98) into master (487f2ee) will decrease coverage by 0.00%.
The diff coverage is 39.70%.

@@            Coverage Diff             @@
##           master   #14758      +/-   ##
==========================================
- Coverage   42.21%   42.20%   -0.01%
==========================================
  Files         767      771       +4
  Lines       81624    82079     +455
==========================================
+ Hits        34458    34644     +186
- Misses      41531    41786     +255
- Partials     5635     5649      +14

Impacted Files
Coverage Δ

models/action.go
48.52% <0.00%> (-2.42%)
:arrow_down:

models/migrations/migrations.go
2.59% <ø> (ø)

models/migrations/v166.go
0.00% <0.00%> (ø)

models/migrations/v172.go
0.00% <0.00%> (ø)

models/migrations/v173.go
0.00% <0.00%> (ø)

models/session.go
0.00% <0.00%> (ø)

models/user.go
53.05% <ø> (+0.38%)
:arrow_up:

modules/context/context.go
65.22% <0.00%> (+6.72%)
:arrow_up:

modules/indexer/code/elastic_search.go
1.72% <0.00%> (-0.02%)
:arrow_down:

modules/migrations/github.go
77.77% <0.00%> (-1.99%)
:arrow_down:

... and 61 more

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 b56c19d...0ccdc98. Read the comment docs.


You don't have permission to access field "role" in collection "directus_permissions" or it does not exist. Queried in root.

Describe the Bug
I believe since version 11 of Directus the data model has changed. directus bootstrap results in a database where directus_permissions has no role column anymore, hence exporting or importing with this tool doesn't work anymore.
To Reproduce
npx directus bootstrap # on a fresh instance with a fresh database
directus schema-sync export # throws

Version
3.0.1
Installed Extension Via
NPM Custom

It still works, although you need to update the schema-sync/directus_config.js file with the following first:
Note: Make sure to replace the two existing collections, and then add the others, in this order.
…
	directus_roles: {
		watch: ['roles'],
		linkedFields: ['parent'],
		query: {
			sort: ['name'],
		},
	},
	directus_policies: {
		watch: ['policies'],
		query: {
			sort: ['name'],
		},
	},
	directus_permissions: {
		watch: ['permissions', 'collections', 'fields'],
		excludeFields: ['id'],
		getKey: o => `${o.policy}-${o.collection}-${o.action}`,
		query: {
			sort: ['policy', 'collection', 'action'],
		},
	},
	directus_access: {
		watch: ['access'],
		excludeFields: ['id'],
		getKey: o => `${o.role ?? o.user ?? 'public'}-${o.policy}`,
		query: {
			sort: ['role', 'policy'],
			filter: {
				user: { _null: true },
			},
		},
	},
	…

@u12206050 You are absolutely right and I feel a bit embarrassed. I guess it was a little bit late yesterday when I tried to update everything and reimport.
Obviously, your tool offers all the necessary routines, but it's the directus_config.js / schema.json that defines the DB's structure and table API. I obviously have to adapt it to the migrations of Directus themselves. It's a bit of a task to work that out, but I guess that inevitable every major version of Directus.
I love your extension, please keep it up. ❤


Jarod/landing page

Description:
Introducing a Landing Page and Updated URI Connection
Changes Proposed:

Addition of a landing page to the application.
Modification of the URI connection to be undefined until manually configured.

Reasoning:
The landing page provides a better user experience, and the updated URI connection allows for manual configuration as needed.
Additional Notes:

The landing page is designed to improve the application's user interface.
The change in URI connection offers flexibility for configuration.

Testing Strategy:

Local testing has been successfully performed to validate the landing page and URI connection changes.
Ensure the server is running locally before testing.

Impact Assessment:

The addition of the landing page enhances user experience.
The updated URI connection offers flexibility for custom configuration.

Checklist:
[x] Landing page added.
[x] URI connection modified for manual configuration.
[x] Code formatting and documentation adhering to established standards.
[x] Successful local testing and verification of changes.
Reviewer Request:
Kindly review the proposed changes for the landing page and URI connection and provide feedback for any improvements or suggestions.
Contributors
@margarethatch
@jennyouk
@JarodCrawford
@alexpalazzo

@JarodCrawford great job. Can't wait to present this mvp tomorrow!


Documentation/op-guide: update container.md

ACI is deprecated, so maybe drop rkt coreos.com/etcd:v3.1.2 commands.
Clean up other commands as well.
https://github.com/coreos/etcd/blob/master/Documentation/op-guide/container.md

/cc @spzala this is mainly a documentation issue. maybe you can help when you read though the doc?

@xiang90 thanks much and yes absolutely, this will be good start :-). I will be working on it soon get back to home tomorrow.

Update on work - started digging into it. Thanks!


Could you share the pytorch implementation?

Really great work.
I am wondering whether have you implemented your method with Pytorch.
It would be great if you could share your method on other platform.

Sorry, we do not have a pytorch implementation for this work.


Configuration: synchronise the variables on the build file templates

For some reason, we didn't use some of the possible target attributes
in the Unix Makefile template, and there was a similar but much
smaller lack of use in the Windows makefile template as well.

Merged.
d368d9d2e4bce444b3c56db55aab5f8450814209 Configuration: synchronise the variables on the build file templates

I don't have permission to re-open this.  Ping @levitte, @openssl/comitters, whoever.  This does not work.  See https://api.travis-ci.org/v3/job/635494784/log.txt and look for OPENSSL_BUILDING_OPENSSL it shows up once.  It should show up in every compilation.

Er, you do remember what -s in make - s does, right ?

yeah, maybe that's not the issue.  never mind.

@levitte, Look at https://ci.appveyor.com/project/openssl/openssl/builds/30049497/job/e17342lihomp389e#L1161  That doesn't appear to be "make -s" and I don't see the define present.

Ah, figured it out, the definition was a bit misplaced.  However, now that I'm looking at it again, that definition should be attached to the public libraries we build, and is actually better placed in the dop build.info.
PR coming up!

See #10813


Create merge-and-de-duplicate-the-array.md

This is a duplicate of https://github.com/phuoc-ng/1loc/blob/master/collections/array/get-union-of-arrays.md

That's true @robinpokorny, this pull request can be closed !


Remove unnecessary quotation for passin and passout

Pull Request (PR) description
Remove unnecessary single quotes for passin and passout parameter.
Also fixes a bug, where the quotation is interpreted as a part of the password

I think the purpose of the single quotation marks was to guard against shell metacharacters being in the password, since this is passed to a command line. What about using https://www.puppet.com/docs/puppet/latest/function.html#shellquote instead?

Tested the latest commit on my deployment, seems to work fine.


Dash Live stream Error

Have you read the FAQ and checked for duplicate issues:
Yes
What version of Shaka Player are you using:
2.1.4
Can you reproduce the issue with our latest release version:
Yes
Can you reproduce the issue with the latest code from master:
Yes
Are you using the demo app or your own custom app:
Both
If custom app, can you reproduce the issue using our demo app:
yes
What browser and OS are you using:
Chrome, Firefox on Windows 10
What are the manifest and license server URIs:
(you can send the URIs to<EMAIL_ADDRESS>instead, but please use GitHub and the template for the rest)
I'send it with mail.
What did you do?
We teste the url and i have this error. This live stream works with dashif player
streaming_engine.js:1103 (audio:5) cannot find segment: currentPeriod.startTime=0 lookupTime=49926.84640002251
shaka.media.StreamingEngine.lookupSegmentPosition_ @ streaming_engine.js:1103
shaka.media.StreamingEngine.getSegmentReferenceNeeded_ @ streaming_engine.js:1055
shaka.media.StreamingEngine.update_ @ streaming_engine.js:944
shaka.media.StreamingEngine.onUpdate_ @ streaming_engine.js:840
streaming_engine.js:1103 (video:12) cannot find segment: currentPeriod.startTime=0 lookupTime=49927.84739995003
shaka.media.StreamingEngine.lookupSegmentPosition_ @ streaming_engine.js:1103
shaka.media.StreamingEngine.getSegmentReferenceNeeded_ @ streaming_engine.js:1055
shaka.media.StreamingEngine.update_ @ streaming_engine.js:944
shaka.media.StreamingEngine.onUpdate_ @ streaming_engine.js:840
streaming_engine.js:1103 (audio:5) cannot find segment: currentPeriod.startTime=0 lookupTime=49927.84739995003
shaka.media.StreamingEngine.lookupSegmentPosition_ @ streaming_engine.js:1103
shaka.media.StreamingEngine.getSegmentReferenceNeeded_ @ streaming_engine.js:1055
shaka.media.StreamingEngine.update_ @ streaming_engine.js:944
shaka.media.StreamingEngine.onUpdate_ @ streaming_engine.js:840
streaming_engine.js:1103 (video:12) cannot find segment: currentPeriod.startTime=0 lookupTime=49928.84840011597
shaka.media.StreamingEngine.lookupSegmentPosition_ @ streaming_engine.js:1103
shaka.media.StreamingEngine.getSegmentReferenceNeeded_ @ streaming_engine.js:1055
shaka.media.StreamingEngine.update_ @ streaming_engine.js:944
shaka.media.StreamingEngine.onUpdate_ @ streaming_engine.js:840

This is a bug on our part.  The time in the availabilityStartTime should default to UTC.  But we simply pass it to Date.parse(), which uses the local timezone.
It would be preferable for your assets to explicitly pass the timezone to avoid errors like this.  For example, use 2017-06-20T03:30:18Z.

The fix for this has been cherry-picked and will be released in v2.1.5.


Added monaco editor instead of text areas for sql pretty and graph ed…

…it dialogs. Fixes #282

Coverage remained the same at 4.941% when pulling 6890f26ed9823baacd951bbb475a99cb93729245 on monaco-editor into 929ab6cb1914ef0d58911945467f29aaaa24c52a on master.


Quieting logs

This might be more a doc issue than a code issue but...
The message ActiveJob gives when it does not enqueue something because of non-uniqueness is pretty nasty (Failed enqueuing XXX to Sidekiq(default), a before_enqueue callback halted the enqueuing execution.). Is there a way to silence that from the logs?

You could pass in a proc that does nothing: https://github.com/veeqo/activejob-uniqueness/blob/main/lib/generators/active_job/uniqueness/templates/config/initializers/active_job_uniqueness.rb#L18

@bmulholland - sorry to not reply earlier. I missed your reply. We actually already tried that. Do you see something I am missing in our syntax?
  unique :until_and_while_executing,
    lock_ttl: 1.hour,
    on_conflict: ->(job) { logger.warn "Already in queue - not queueing again: #{job.arguments}" },
    on_runtime_conflict: ->(job) { logger.warn "Already executing - not queueing again: #{job.arguments}" }

Note that we tried it empty and had no output, and just on a whim moved it to this where we would at least see something if it started running the procs.
Here is an example of the lines we still get in our logs:
[ActiveJob] [CacheWarmingJob] [7aafce65-1b3a-4183-bb44-ba447fb77cd1] Error performing CacheWarmingJob (Job ID: 7aafce65-1b3a-4183-bb44-ba447fb77cd1) from Sidekiq(low_priority) in 78.73ms: a before_perform callback halted the job execution

I should also call out that I know this is ActiveJob itself doing this - in fact I know the line of code is https://github.com/rails/rails/blob/13b3e7d83988d451ea305243319cc715a06617ea/activejob/lib/active_job/log_subscriber.rb#L93. But I was hoping there was an established wait to silence this would either a hacked logger or monkeypatching that subsciber.


More verbose debug logging on decode error

Right after that last PR #154 I realized I forgot about debug logging.

Or I'm just to quick on the merge button!  😆   Good catch.  Thanks again for your PRs.


fix: don't remove finalizer of IstioOperator when action is still required

Please provide a description of this PR:
This issue helps resolve #35016 when deleting the IstioOperator ingress/egress components are not cleaned up. This seems to be an issue when the helm reconciler sets a status of InstallStatus_ACTION_REQUIRED. The code currently does not seem to check that here and instead removes the finalizer immediately not giving the user the chance to understand why the components weren't removed.

Hi @strantalis. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@googlebot I signed it!

@strantalis: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Add credential profile in used by the aws_lambda filter instance from the AWS credentials file.

Commit Message: aws_lambda: add credential profile config to be used with AWS credentials file.
Additional Description: Adds a new config to allow a filter instance use an specific profile from the AWS Credential File.
To implement this signer needs to be moved into the config object to be able to honor per route configuration.
Because the needs of this change It was decided to follow aws_request_signature approach. doing this will make easier in the future to adopt SigV4a.
Related: https://github.com/envoyproxy/envoy/issues/31403
Risk Level: Low
Testing: yes
Docs Changes: yes
Release Notes: yes
Platform Specific Features: n/a

cc @suniltheta


Drive Girls [PCSE01099]

App summary

App name: Drive Girls
App serial: PCSE01099
App version: 1.00

Vita3K summary

Version: v0.1.6
Build number: 3101
Commit hash: https://github.com/vita3k/vita3k/commit/[8623b51]
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: JackMonty
OS: Windows 10
CPU: AMD Ryzen 5 Pro 3500U
GPU: Nvidia Geforce GT1030
RAM: 16 GB

Issues
Game runs fine with good speed and audio in Vulkan renderer. In game engine (fight scenes) also works well. Do not use OpenGL it gives lots of glitches when playing in the in game engine. Issues with background images not appearing during title screen, menus and tutorials. Menus themselves work okay and you can still select everything. Game is playable except for lack of backgrounds - issues occurs in both OpenGL and Vulkan.
Screenshots
Missing Background (Title Screen)

Missing Background (Mission Start)

Tutorial not displaying:

Story Segment (shows background fine)

Fight Scenes working well

Log
vita3k.log
Recommended labels

Playable
Graphics Bug

Seems to be a regression as a previous build on the same game but different ID [PCSB01097] has background images working.
Same issue experienced in: https://github.com/Vita3K/compatibility/issues/1433

Hi @resaaa2023 ,
Seems to work beyond that for me... The you have the logs for when you get the issue? If yes please add them and the version you are using.

RETESTED ON:
Vita3K summary

Version: v0.1.8
Build number: 3157
Commit hash: https://github.com/vita3k/vita3k/commit/[4280bea5]
CPU backend: Dynarmic
GPU backend: Vulkan

Now fully working
Title Screen

Mission Select

Tutorial

Recommend remove graphic bug and close issue. I could not replicate crashing issue mentioned by @resaaa2023.