Provide example of some rules and matches, especially with spaces

It would be nice to provide some examples of matches and rules, especially with a space in involved, as well as some more complex matching patterns.
We had the need to perform a redirect from our CDN to another URL for a file that we no longer wanted to be published. There was no file on the backing Blob store, but we put in a rule to match on URL and then redirect to a different page. The issue we ran into was the file had spaces in the name, for example, "downloads/product user guide.pdf". Browsers will encode this to be "downloads/product%20user%20guide.pdf". The docs specifically call out that % is used for URL encoding, and specifically uses %20 as an example. This led us to using "downloads/product%20user%20guide.pdf" as part of the match criteria; however, it appears that it should have been "downloads/product user guide.pdf". This ended up taking a lot of time to figure out what the match wasn't working since there is almost zero troubleshooting capability on the matches.
The recommendation here would be to provide three to four different examples of the matching, especially with spaces and some of the wildcard characters.

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

ID: a2dc930c-33a3-d54e-2253-1b1b5a693c17
Version Independent ID: 7dc5aee5-f473-10c8-5299-d5d0b52b7077
Content: Standard rules engine reference for Azure CDN
Content Source: articles/cdn/cdn-standard-rules-engine-reference.md
Service: azure-cdn
GitHub Login: @asudbring
Microsoft Alias: allensu

@mikewo we'll review your feedback and get back to you shortly. Thanks.

@mikewo , thank you for your suggestion. We have assigned this to the content owner/author for review.
@asudbring , could you please review the suggestion and provide your inputs?

Assigning to PM for review:
#reassign:@sohamnchatterjee

Assigning to PM for review
#reassign:@amitsriva

#reassign: gunjanj

Thank you for you dedication to our documentation.
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


No support for dark mode in Chrome

In Chrome, a developer can set the theme to dark mode. Augury should display in dark mode as well.

@steveblue what are you using for the dark theme?

The dark mode built into chrome dev tools settings. It was released in Chrome Canary a couple months ago and is now available in Chrome builds.

@steveblue thats for the suggestion! We will consider having dark theme in the future.

I don't think there currently is a programatic way to find out whether Chrome is in dark mode or not.

@vanessayuenn yeap I tried that already it changes the class on the window but in Augury opens in an iframe so it doesn't have access to parent window

Yup so this is deferred for now. If anyone wanna take a crack at it, feel free to do so.

This feature is awesome! @sumitarora @danielschifano @vanessayuenn @igor-ka TY very much for implementing this.
I can finally feel like I'm in stealth mode!


Game Crashes Before Character Select Screen

So I downloaded the latest Version of the mod here: https://ci.appveyor.com/project/SonicFreak94/sadx-dc-lighting/build/artifacts
After installing it, i ran my game and it crashed everytime I tried to go to the character select for example Adventure Mode.

It looks like AppVeyor skipped commit 16795b9f492738cfbb8f33f0be94806593033a6c
It's crashing because I updated the d3d8to9 submodule to be compatible with the latest release version. Grab that from its repo (linked in README.md) and it should work fine.

sorry but I can't find the README.md

Now it works, thanks!


Xcode failing to build for simulator

Describe the bug
Xcode build fails when trying on Simulator
This Bug is About

[X] Integration

Platform

[X] iOS

Environment
Please tell us the versions of SDKs you are using. If you are not sure about React Native version, you can run react-native --version under your project.

React Native version: 0.63.2
react-native-flurry-sdk version: 6.1.0

To Reproduce
Steps to reproduce the behavior:

From XCode Select a Simulator and build

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Are you using Flurry Push/Messaging?
No
(iOS) Are you using CocoaPods for React Native integration?
Yes
ld: in /ios/Pods/Flurry-iOS-SDK/FlurryConfig/libFlurryConfig_11.1.0.a(libFlurryConfig.a-arm64-master.o), building for iOS Simulator, but linking in object file built for iOS, file '/ios/Pods/Flurry-iOS-SDK/FlurryConfig/libFlurryConfig_11.1.0.a'
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Hi @chrisid, are you using xcode12?

Hi @chrisid, are you using xcode12?

Yes, I am. I'm sorry I forgot to mention that.

@chrisid, we are going to release binary build with xcode12 which will resolve this issue. Current SDK doesn't support xcode12 build. Sorry for this inconvenience.

@chrisid We have released an xcode12 compatible build v6.1.9. Please try to use it with xcode12. Thanks!
https://www.npmjs.com/package/react-native-flurry-sdk/v/6.1.9

Closed because fix is available in release v6.1.9.

Closed because fix is available in release v6.1.9.


Two-mile difference between car speedometer and OpenPilot

Describe the bug
The max speed setting on OpenPilot is always two miles below that displayed on my car. The real-time speed on OP is 1-2 miles/hour below my car's shown speed.
How to reproduce or log data
Driving on a highway, engage car cruise control, engage OP and set desired max cruise speed.
Expected behavior
I expect the max set speed to be the same on OP and my car's display.
Device/Version information (please complete the following information):

Device: Comma Two
Dongle ID: cd21d47a9d46bdf3
Route: cd21d47a9d46bdf3|2021-05-15--09-02-28
Timestamp: Entire route
Version: 0.8.3
Car make/model: Toyota Corolla Hybrid 2021

Additional context
None at this time.

openpilot's speed is directly read from the wheels. The speedometer fudges the speed a bit to ensure that the displayed speed isn't lower than the actual speed. Currently, we've chosen to use the actual speeds, but we understand this is confusing and may revisit this in the future.


feat(//core/partitioning) : Dynamic shapes + fallback

Description
Dynamic shapes + fallback. This is a refactor of https://github.com/pytorch/TensorRT/pull/1111 (credits to @Njuapp).
Type of change

New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project (You can use the linters)
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas and hacks
[ ] I have made corresponding changes to the documentation
[x] I have added tests to verify my fix or my feature
[x] New and existing unit tests pass locally with my changes
[x] I have added the relevant labels to my PR in so that relevant reviewers are notified

Make strings like min, opt, max as enums


Dbeaver adds 1 year to date column result

System information:

Operating system (distribution) and version
Windows 10 Pro
DBeaver version
5.1.5

Connection specification:

Database name and version
MS SQL Server

Describe your question:
I use Dbeaver to query result from MS SQL Server tables.
I just noticed some strange in result of the table which contain date column.
The date '2020-12-30' is shown in result, while the value is actually '2019-12-30' (I tested this by using bcp to get data from SQL Server, and also tried to query with where date = '2020-12-30', no result)
Unfortunately, I cannot reproduce more cases due to company's restriction to access data tables and read only permission.
However, as I mentioned that I tried using bcp to get data from server and date is '2019-12-30', so I just guess that there is something with Dbeaver.
Could someone suggest what's actually happening with the result, or how should I deal with it?
Thank you.
Result in DBeaver

Result from BCP

Unfortunately cannot reproduce it. Could you please make a screenshot of "Data Formats" page (File>Properties>Editors> Data Editor> Data Formats)?
Also you can try the latest version - 6.3.5 (https://dbeaver.io/download/).

Unfortunately cannot reproduce it. Could you please make a screenshot of "Data Formats" page (File>Properties>Editors> Data Editor> Data Formats)?
Also you can try the latest version - 6.3.5 (https://dbeaver.io/download/).

Please refer to below.

Unfortunately, I cannot install other versions due to (company's policy -- again :( ).
However, if it can be proved that the problem come from this version, that may be the case.

ticket will be closed. feel free to reopen an issue if it is reproducible for you against recent version


[Model] Flora

Odd that this is named Flora when both the model and the zip file are named Beatrice - it appears to be no different to the 2 other packages for this model except that one of them is a ut2mod.

URL: https://unrealarchive.org/models/unreal-tournament-2004/F/flora_0d06c042.html
Hash: 0d06c042f9e0db3a07aa03237c4f98d4081dfa1f
Current name: Flora

That is weird.
Also flagged as variation of the other one.


add indication of file change in buffer

Support added on the commit https://github.com/adelarsq/neoline.vim/commit/ff3d8af14ce2e157759b0f09d3c674ce99e2cd32
The idea is to see all file changes on the same place.
Thanks @rustiever

Support added on the commit https://github.com/adelarsq/neoline.vim/commit/ff3d8af14ce2e157759b0f09d3c674ce99e2cd32
The idea is to see all file changes on the same place.
Thanks @rustiever


[SIL] Allow specify_test instructions to take value literals.

Tests can now say %foo to refer to a value in addition to @instruction or @argument or similar.

@swift-ci please test


Fix calendars extra

I noticed that pip install dateparser[calendars] didn't work. After some time investigating, it seems that there was a typo in extras_require.
I also updated the tox file after this change and the docs (that were wrong). The requirements.txt file will be deleted in this PR: https://github.com/scrapinghub/dateparser/pull/739

I added a temporary workaround (pinning a working regex package version)  to make the Pypy pipeline pass.
As mentioned here: https://github.com/scrapinghub/dateparser/issues/565, it seems that dateparser doesn't work with Pypy out of the box because of the regex library which we depend on.
The next version will remove Python 2.7 support: https://github.com/scrapinghub/dateparser/issues/727 (and Pypy), so I hope we will remove this requirement pinning before releasing the next version.
I will investigate if this affects pypy3 to know if we can support it or not.


Add link to IBM SoftLayer instructions

This is a continuation of PR 1183 (https://github.com/kubernetes/kubernetes.github.io/pull/1183), which I messed up when I rebased.
--- Original comments:
Originally, I created a page describing the turn-key solution to deploy Kubernetes in IBM SoftLayer, but the reviewer on the following PR #313 asked me to host the information in another repository.
So this PR is to create a link to the external repository containing the instructions.
Thanks!

This change is

@devin-donnelly I need to create a new PR because I messed up during rebase.

LGTM. Thanks. Is this ready for immediate publication? If so, I'll merge.

Yes, it is. Thanks for doing it!

@patrocinio maybe you'd want to update the table of solutions too?

@janetkuo Good point... I'll update the table of solutions. I thought they were not for turn-key ones, but I see GCE there.
I'll work on it in another PR.


Facebook Blocked from NPM Startup?

Hi,
Been running miscord without a problem for a while now. This afternoon, I got a message that NPM couldn't log into Facebook, perhaps for being blocked from logging in from an unknown location. Logging in from a browser, there was indeed a security block. I verified the account, but even after doing so and being able to log back on in the browser, NPM still can't log in... any idea as to how I might get around this?

As an update, I created a new Facebook account in hopes it could get around it, but ran into same issue... some sort of security change? Hoping there's a solution, the bridge has become very important to us...

Currently we're having different issues on Facebook's side, I hope we'll get to bottom of this quickly.

Desperate to get this working, I had the bright idea to use w3m to log into the facebook account, approve and remember the device for the unknown login attempt, and try it again.  Though the w3m login and subsequent alert / approval worked like a charm, I am STILL getting the same error from npm.
SO, I tried renaming appstate.json, as that has helped with other problems in the past.  When I do, I get this different error that I cannot explain:
info login Logging in to Facebook...
ERR! login type can only be one of the following: inbox, pending, archived, other
ERR! error { error: 'type can only be one of the following: inbox, pending, archived, other' }
...what is "type"? Doesn't seem to be a config issue, unless I'm missing a property in there...

I seem to be running into this issue as well. Facebook just never remembers the device. Account just gets a temp block every time I run the bot so it can never actually connect.

@peteralynn it looks like you're using newer version of Miscord with older version of facebook-chat-api... can you re-download dependencies?
@XerockXMG try logging in with a browser and refreshing the notifications while starting Miscord, at some time you would get a request to verify a new browser so just accept it before Miscord errors outz

Updating solved my problem--so it is good practice to run "npm update" rather than pulling from git, as I usually do?

@peteralynn yeah, npm update gives me the best results.


introduce php56 version

based on work originally done by @remicollet here: https://github.com/openshift/sti-php/pull/42

@mfojtik ptal again.

@bparees LGTM (few small nits)

@mfojtik i think i addressed all the comments...currently failing because software collections is down, so i can't get far enough to see if there are any other issues.

[test]

LGTM

[test]

[test]


Add TSC's FileSystem implementation on top of CASTree

could we add a couple tests for this implementation? something for the more complex methods like the contents of directories and files

this will indirectly have tests because swiftpm-on-llbuild2 will consume it but you’re right that it’ll be worth having some dedicated unit tests as well...


require-error: minimize the number of false positives

Closes
https://github.com/Antonboom/testifylint/issues/5

@scop @rliebz @max-melentyev @AleksSi
Hi, guys!
Thank you for interest to my project.
Could you please run
go install github.com/Antonboom/testifylint@b6f56e0
testifylint --enable require-error ./...

And give me feedback about warnings?
Is it ok now?

@rliebz thank u for such detailed feedback. Appreciated.
Based on my experience assert for errors is suitable in rare cases (the most part you described), usually presented in large or specific code bases. Otherwise this make testing debug annoying during panics, and messages from subsequent assertions usually make no sense or you don’t pay attention to them.
For me this is similar to ignoring errors in primary code:
v, _ := operation()

I made some googling and found support of my position, e.g. this (but this is really an exaggerated example).
I tested require-error checker (as linter as well) on several the biggest go projects and in most cases warnings were valid.
The rest false positives really to hard to analyze and I agree with you.
I'll leave the checker enabled by default, waiting for a new wave of feedback from the community.
Also as a result of our discussions I plan to add two new checkers (graceful-teardown and go-require), I will keep you updated.
Thanks!


Create Assets

folder creation
a folder to hold image and sound assets for games and the arcade machine

just so future teams can access and grab assets found or made by the current students with references to the source if found
this is just so that things like sound assets that have already been found don't have to be searched for again as it can be very time consuming referencing all of them and finding the right sound. Also, this folder would allow Lily to commit her work towards finding these sounds. as they are currently stored only on teams in Applications Team files.


Added sample script that writes to Snowflake.

Description of changes:
Added sample script that writes to Snowflake. Snowflake jar files are commented out in download_jars.sh
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Addition of Snowflake connector and Glue integration


[Documentation] "Ambient scene" makes no sense at all

"Ambient scene" here:
https://github.com/ytsaurus/ytsaurus/blob/74c5ce917e8bb59843df5dc6684e43e59d3f369c/yt/docs/en/_includes/user-guide/data-processing/porto/layer-paths.md?plain=1#L9
makes no sense at all.

Hi! Could you, please, describe what you want to see here? Or what is leading to misunderstanding?


the README coerce_key example doesn't work like expected

require 'hashie'

class User
  def initialize(*attrs)
    require'pry-byebug';binding.pry
  end
end

class Tweet < Hash
  include Hashie::Extensions::Coercion

  coerce_key :user, User
end

# t = Tweet.new
# t[:user] = { name: 'Vasya' }
# ^--- this way it works

t = Tweet.new(user: { name: 'Vasya' })
# ^--- doesn't work when using initializer

❯ ruby -v
ruby 2.2.3p173 (2015-08-18 revision 51636) [x86_64-darwin15]

The same with collections example:
require 'set'
require 'hashie'

class User
  def initialize(*args)
    puts args
    require'pry-byebug';binding.pry
  end
end

class Tweet < Hash
  include Hashie::Extensions::Coercion

  coerce_key :mentions, Array[User]
  coerce_key :friends, Set[User]
end

user_hash = { name: 'Vasya' }

mentions = [user_hash, user_hash]
friends = [user_hash]

t = Tweet.new(mentions: mentions, friends: friends)
# ^-- doesn't work
require'pry-byebug';binding.pry

t[:mentions] = mentions
# ^-- works as expected

puts t

This is just a hunch but I think it's because the constructor doesn't behave how people expect it to, unless you've mixed in MergeInitializer as well. I'll check it out and report back.

@michaelherold is for sure correct about this. You must include Hashie::Extensions::MergeInitializer to enable coercion in the initializer.
require 'hashie'

class User
  def initialize(*attrs)
    require'pry-byebug';binding.pry
  end
end

class Tweet < Hash
  include Hashie::Extensions::Coercion
  include Hashie::Extensions::MergeInitializer   # Add me here!

  coerce_key :user, User
end

# t = Tweet.new
# t[:user] = { name: 'Vasya' }
# ^--- this way it works

t = Tweet.new(user: { name: 'Vasya' })
# ^--- __it works!__

Hah, thanks for the update. I totally looked into this, but apparently did not remember to report back.
I don't think this is something that needs to be added to the README. The initializer is a common misconception, but it's a common misconception with base Hashes as well.
Does anyone else have thoughts on that?

Going back to the original problem, and without looking, if the example in the README doesn't work as described it should be fixed!


Missing scale configs?

Hi, team!
I am trying to run the training code, but it seems scale config files are missing. (https://github.com/mlfoundations/dclm/blob/fbb24afbc66abed64b5fefc16f12f3511138f05e/training/hyperparameters.py#L81)
Are these config files going to be uploaded later, or do we have to write it ourselves?

sorry about that! we just merged the configs used for the competition scales.


Add --dns=none

Fix for #2744
This adds support for --dns=none, which specifies that we should not bind-mount in /etc/resolv.conf.
Still to be added: similar flag for disabling addition of /etc/hosts (can't reuse the existing flag). Also needs tests.

@rhatdan @baude Thoughts on naming for a new flag to disable creation of /etc/hosts?

could --dns=none work for both /etc/resolv.conf and /etc/hosts or are there cases where we would like one of them but not the other?

I am fine with the option name.
Needs update for man pages.

Added a new flag --no-hosts to disable creation of /etc/hosts. Added manpages and bash completions. Removing WIP. Working on tests now.

Testing this is proving to be a pain, because I don't want to tie us to the contents of any specific image's /etc/hosts and /etc/resolv.conf (we can't manually add them with buildah because they're being mounted over, and Docker and Podman both previously overwrote them unconditionally so nobody really cared what was in them)

Updated to address review comments

LGTM

bot, retest this please

/lgtm


Hardcode missing headers to build in 2023.0

As is:
/victor@accelerationrobotics:/media/victor/intel/oneAPI-samples/DirectProgramming/C++SYCL_FPGA/Tutorials/Tools/experimental/platform_designer/add-oneapi/build$ cmake .. -DFPGA_DEVICE=agilex
-- The CXX compiler identification is IntelLLVM 2023.0.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /opt/intel/oneapi/compiler/2023.0.0/linux/bin/icpx - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring the design to run on FPGA board agilex
-- Additional USER_FPGA_FLAGS=
-- Additional USER_FLAGS=
-- Additional USER_INCLUDE_PATHS=../../../../../include
-- Additional USER_LIB_PATHS=
-- Additional USER_LIBS=
-- Configuring done
-- Generating done
-- Build files have been written to: /media/victor/intel/oneAPI-samples/DirectProgramming/C++SYCL_FPGA/Tutorials/Tools/experimental/platform_designer/add-oneapi/build
victor@accelerationrobotics:/media/victor/intel/oneAPI-samples/DirectProgramming/C++SYCL_FPGA/Tutorials/Tools/experimental/platform_designer/add-oneapi/build$ make fpga_ip_export
[ 33%] To compile manually:
/opt/intel/oneapi/compiler/2023.0.0/linux/bin/icpx -I../../../../../../include -fsycl -fintelfpga -Wall -qactypes -DFPGA_HARDWARE -c ../src/add.cpp -o CMakeFiles/fpga_ip_export.dir/src/add.cpp.o

To link manually:
/opt/intel/oneapi/compiler/2023.0.0/linux/bin/icpx -fsycl -fintelfpga -Xshardware -Xstarget=agilex -fsycl-link=early -fsycl-device-code-split=per_kernel -o add.fpga_ip_export CMakeFiles/fpga_ip_export.dir/src/add.cpp.o
[ 33%] Built target displayExportCompileCommands
[ 66%] Building CXX object CMakeFiles/fpga_ip_export.dir/src/add.cpp.o
/media/victor/intel/oneAPI-samples/DirectProgramming/C++SYCL_FPGA/Tutorials/Tools/experimental/platform_designer/add-oneapi/src/add.cpp:10:10: fatal error: 'sycl/ext/intel/prototype/host_pipes.hpp' file not found
#include <sycl/ext/intel/prototype/host_pipes.hpp>
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
make[3]: *** [CMakeFiles/fpga_ip_export.dir/build.make:76: CMakeFiles/fpga_ip_export.dir/src/add.cpp.o] Error 1
make[2]: *** [CMakeFiles/Makefile2:204: CMakeFiles/fpga_ip_export.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:211: CMakeFiles/fpga_ip_export.dir/rule] Error 2

After the fix:
victor@accelerationrobotics:/media/victor/intel/oneAPI-samples/DirectProgramming/C++SYCL_FPGA/Tutorials/Tools/experimental/platform_designer/add-oneapi/build$ cmake .. -DFPGA_DEVICE=agilex
-- Configuring the design to run on FPGA board agilex
-- Additional USER_FPGA_FLAGS=
-- Additional USER_FLAGS=
-- Additional USER_INCLUDE_PATHS=../../../../../include
-- Additional USER_LIB_PATHS=
-- Additional USER_LIBS=
-- Configuring done
-- Generating done
-- Build files have been written to: /media/victor/intel/oneAPI-samples/DirectProgramming/C++SYCL_FPGA/Tutorials/Tools/experimental/platform_designer/add-oneapi/build
victor@accelerationrobotics:/media/victor/intel/oneAPI-samples/DirectProgramming/C++SYCL_FPGA/Tutorials/Tools/experimental/platform_designer/add-oneapi/build$ make fpga_ip_export
[ 33%] To compile manually:
/opt/intel/oneapi/compiler/2023.0.0/linux/bin/icpx -I../../../../../../include -fsycl -fintelfpga -Wall -qactypes -DFPGA_HARDWARE -c ../src/add.cpp -o CMakeFiles/fpga_ip_export.dir/src/add.cpp.o

To link manually:
/opt/intel/oneapi/compiler/2023.0.0/linux/bin/icpx -fsycl -fintelfpga -Xshardware -Xstarget=agilex -fsycl-link=early -fsycl-device-code-split=per_kernel -o add.fpga_ip_export CMakeFiles/fpga_ip_export.dir/src/add.cpp.o
[ 33%] Built target displayExportCompileCommands
[ 66%] Building CXX object CMakeFiles/fpga_ip_export.dir/src/add.cpp.o
[100%] Linking CXX executable add.fpga_ip_export
[100%] Built target fpga_ip_export

I'm closing this PR as the 2023.0 as been removed now that 2023.1 is out.


Can't get this to work

Hey! I'm really struggling to get this plugin working, I'm hoping someone can help me figure this out.
I had originally installed tpope's copilot.nvim and authenticated through that plugin, and had it working showing suggestions.
I'm now trying to use this plugin instead, with the hopes of eventually pairing it with copilot-cmp, but I'm not getting any inline suggestions at all.
Here's how I'm setting up the plugin:
  -- Copilot
  use {
    "zbirenbaum/copilot.lua",
    cmd = "Copilot",
    event = "VimEnter",
    config = function()
      vim.defer_fn(function()
        require("copilot").setup({
          copilot_node_command = vim.fn.expand("$HOME")..'/n/bin/node',
        })
      end, 100)
    end,
  }

Here's :Copilot status:

Here's my :LspInfo output, where I noticed filetypes is empty:

I am however able to get suggestions via :Copilot panel:

I'm currently on MacOS, but i've also tried Arch Linux without any luck.

What happens if you run :Copilot suggestion? If that fixes it, you may want to try explicitly enabling suggestions in your config.
Also, are you using copilot-cmp? Right now copilot-cmp and inline suggestions are mutually exclusive.

:Copilot suggestion does nothing, no output.
I don't currently have copilot-cmp installed. I installed it briefly earlier, but it didn't work. Then i noticed i can't even get the inline suggestions to work so i figured i should get that working first?

Ok that did @MunifTanjim , thanks :+1: . I didn't realize it needed to be explicitly set to auto_trigger


Services: first build: abs added

{package} {version} {:snowflake:}
Destination channel: {Snowflake | defaults}
Links

{ticket_number}
Upstream repository
Upstream changelog/diff
Relevant dependency PRs:

...

Explanation of changes:

...

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_9e8za9p6qf/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:19: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_fefylj53bb/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:20: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
clone/recipe/meta.yaml:15: avoid_noarch: noarch: python packages should be avoided
===== Final Report: =====
0 Errors and 2 Warnings were found


user ID does not exist and breaks the download v20210822

Prerequisites

[X] Did you read FAQ section in readme.md?
[X] Did you test with the latest releases or commit ?
[X] Did you search for existing issues in Issues?

Description
First i use version 20210822 from master branch. I have a long download list which i always repeat every 2-3 days, some of artist happened to deactivate their account, eventually reactivate, or some were just banned, not mentioning their artworks often get removed.
I use a new computer and i pulled the master branch, sometimes if the downloader reaches a suspended / not found user, the download breaks.
Steps to Reproduce

download from list
pixiv sketch yes
stumbled upon a dead user
error and breaks the download

Expected behavior: Move on to the next download
Actual behavior: shows this error message and break the download
Error at process_list(): (<class 'PixivException.PixivException'>, PixivException('User has left pixiv or the user ID does not exist.'), <traceback object at 0x0105E788>)
Failed
User has left pixiv or the user ID does not exist.

Versions
20210822

Thanks, I've tried it again, with the exact steps to reproduce, and it gives me this error when it reached a removed account, i think as far as i remember, it is the very same account which caused this issue #1011
Processing Member Id: 2446302
Page
Member Url: https://www.pixiv.net/ajax/user/2446302/profile/all
Member ID (2446302): PixivException(1003 User has left pixiv or the user ID does not exist., hasDumpPage=Y)
User has left pixiv or the user ID does not exist.
Member Url: https://www.pixiv.net/ajax/user/2446302/profile/all
Something wrong, retrying after 2 second (1) ==> PixivException(1003 User has left pixiv or the user ID does not exist., hasDumpPage=Y)
Error at process_list(): (<class 'AttributeError'>, AttributeError("type object 'datetime.time' has no attribute 'sleep'"), <traceback object at 0x01727EC8>)
Failed
Traceback (most recent call last):
  File "D:\Python\PixivUtil2\PixivListHandler.py", line 70, in process_list
    (artist_model, _) = br.getMemberPage(item.memberId)
  File "D:\Python\PixivUtil2\PixivBrowserFactory.py", line 768, in getMemberPage
    artist = PixivArtist(member_id, response, False, offset, limit)
  File "D:\Python\PixivUtil2\PixivArtist.py", line 42, in __init__
    raise PixivException(payload["message"], errorCode=PixivException.OTHER_MEMBER_ERROR, htmlPage=page)
PixivException.PixivException: PixivException(1003 User has left pixiv or the user ID does not exist., hasDumpPage=Y)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "PixivUtil2.py", line 1395, in main
    np_is_valid, op_is_valid, selection = main_loop(ewd, op_is_valid, selection, np_is_valid, args, options)
  File "PixivUtil2.py", line 1091, in main_loop
    menu_download_from_list(op_is_valid, args, options)
  File "PixivUtil2.py", line 478, in menu_download_from_list
    PixivListHandler.process_list(sys.modules[__name__],
  File "D:\Python\PixivUtil2\PixivListHandler.py", line 85, in process_list
    time.sleep(2)
AttributeError: type object 'datetime.time' has no attribute 'sleep'
Unknown Error, please check the log file: (<class 'AttributeError'>, AttributeError("type object 'datetime.time' has no attribute 'sleep'"), <traceback object at 0x01727AA8>)
press enter to exit.

updated, due to wrong import. change to use helper method instead :P

It works perfectly! Thanks a lot! I will post some new issue if i found some during the download run later on


Dont pass account/domainid for upload sslcert in project view

In project view, we can upload ssl certificate either by using
projectid or accountid/domainid combination. Since we are passing
all 3 parameters, we cant upload sslcert in project view

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/893 (JID-3755)

@ravening This will make the form require the user to enter the account as well as domain which is redundant as he's already navigated to the desired account. Instead, would it be better to exclude passing the projectid if the current route contains "account" over at https://github.com/apache/cloudstack-primate/blob/master/src/utils/request.js#L83 ?

@ravening This will make the form require the user to enter the account as well as domain which is redundant as he's already navigated to the desired account. Instead, would it be better to exclude passing the projectid if the current route contains "account" over at https://github.com/apache/cloudstack-primate/blob/master/src/utils/request.js#L83 ?

@davidjumani we don't need to enter anything here. If we are in project view then project id is passed else account/domain will be taken from backend code
If we pass both account and domain in project view then exception is thrown

@ravening The account will be taken for the current user but say an admin will have to pass the account and domain while adding a certificate for another account, after having navigated to that account

Ping @ravening any update on this?

Ping @ravening any update on this?

@ravening @rhtyd @davidjumani
can we do like this ?
(1) in project view, ssl cert will be uploaded to project account, no matter what current account is.
otherwise there is no way to upload ssl cert to a project.
(2) in normal view, ssl cert will be uploaded to the current account on web page.

@ravening @rhtyd @davidjumani
can we do like this ?
(1) in project view, ssl cert will be uploaded to project account, no matter what current account is.
otherwise there is no way to upload ssl cert to a project.
(2) in normal view, ssl cert will be uploaded to the current account on web page.


Setting vim.o.background breaks auto-detection

I'm using a config based on kickstart, which uses lazy.vim. It apparently loads the plugins before vim sets vim.o.background by reading it from the terminal. This means when the vscode plugin gets loaded it reads background's default value of 'dark', and by writing this back to vim.o.background it seems to disable vim's subsequent autodetection, so it always gets forced to 'dark' even if the terminal has a light background.
If I disable the line vim.o.background = ... in the plugin's init.lua, auto-detection works correctly. Please could you consider changing it? The code below seems to work for me, and should also work for anyone who relied on the old behavior.
    local background = style or config.opts.style
    if background and background ~= vim.o.background then
        vim.o.background = background
    end

@realh - Can you make PR if you fixed the issue?


Auto installation

Since most of the time the component's installation is simple and straightforward, it's convenient to be able to install it via a simple import statement:
import 'vue-tabs-component/auto';

I don't see the need of importing the whole Vue library just to register some global components. This adds to the component size if bundled for browsers.
I would prefer a plugin for this situation which doesn't require the Vue dependency
import Tab from './components/Tab';
import Tabs from './components/Tabs';
export default = {
  install (Vue) {
   Vue.component('tab', Tab);
  Vue.component('tabs', Tabs);
  }
}

@cristijora that seems like a good way to go about this
@JosephSilber penny for your thoughts?

Additionally, I'm not a fan of having to specify a path for imports, and prefer default and named exports from a single index file.
@JosephSilber Your PR would currently require users to import vue-tabs-component/dist/auto, which doesn't really feel good to me. @cristijora's strategy would allow uw to use the plugin as the default export without doing any breaking changes either.

This adds to the component size if bundled for browsers.

How so? The default entry point doesn't import this file, so its graph doesn't depend on Vue.

Your PR would currently require users to import vue-tabs-component/dist/auto, which doesn't really feel good to me.

If that's a concern, we can move this file to the top-level directory, like many other projects do.

I'm not a fan of having to specify a path for imports, and prefer default and named exports from a single index file.

Other projects already offer this name/auto, for example: es6-promise.
At the end of the day this is a stylistic choice. Do what feels right to you.

I preferred @cristijora 's way of going about this. I've added his proposal to the package.
Nevertheless thank you @JosephSilber for your input.


Update mockplus.rb

Thank you for the contribution. It needed some fixes, so they were made in 2df42d5e34ff7a17b756c5f82f71979d1a10b77e. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


$stream->on('connection') not triggering on ldap_bind

This is the continue of problem https://github.com/reactphp/stream/issues/48
I updated my gist here https://gist.github.com/tokidoki11/668a3a6f0888c05995b5f45bb2439322
I tried to use ldap_bind, but the process seems to be stuck. The On Connection event is not triggering at all.
What I have done:

Try to ldapsearch via another terminal, it works.. (this only for proof of concept, I need to do this at the same terminal)
Try to make class of thread, and then run the thread on the create callback

create(60000,function()
{
    $thread = new LDAPSearchThread(<ID>);
    $thread->run();
});
class LDAPSearchThread extends Thread
{

    private $searchID;
    private $searchData;

    public function __construct($searchID)
    {
        $this->searchID = $searchID;
    }

    public function run()
    {
        echo "running thread";
        //creaete ldap coonection and bind //stuck also
    }

}

It's weird though, if I invoke fsockopen, the connection event will trigger.
If I invoke fsockopen then ldap_bind, the connection event will not trigger

@clue maybe you could give me an advise? :)

I'm not sure I follow you code, so perhaps you may help by reducing this to the bare minimum to highlight the issue you're seeing?
As a general rule of thumb, anything that is blocking (such as your LDAP query) does not work well with React. Unless you feel like implementing an async LDAP client on top of React PHP (which I'd love to see), I'd suggest moving the blocking parts to a separate process and using the ChildProcess component for IPC.

Basically i want to query ldap via the proxy server i made on port 60000.

Yes, also see https://github.com/reactphp/react/wiki/FAQ#how-can-i-use-blocking-functions
You may get away with blocking for small periods of time (such a local fsockopen), but these are generally considered experimental and unsupported and better alternatives usually exist, such as the SocketClient in this case.

I guess I cant do much....


Add examples for bio acoustic sonar and cava cloud

Overview
Adds more examples on how to use yodapy.

Pull Request Test Coverage Report for Build 356

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

Totals

Change from base Build 354:
0.0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls


Fix size calculation of mapping returned by split_to.

The returned mapping had a range of [0, size) instead of [0, at).

Good catch. Thank you for the PR!


Testing protocol for #72

#72
To repro

Open VSCode
Install latest vsix
create new evidence-app folder
go to extensions tab
Evidence is already installed
?

Resolved


Increase test coverage.

Increase coverage for TaskList, AddCommand, DeleteCommand, ModifyCommand, ListCommand, FindCommand.

Fixed by PR #106


Babylon 101 Video Tutorial Series

Create a series of tutorial/walkthrough videos to accompany the Babylon101 documentation.
These videos will help walk folks who are new to Babylon, through the beginning basics of how to use it.

[ ] Intro and About the Course
[ ] First Steps
[ ] Discovering the Basics
[ ] Parametric Shapes
[ ] Position and Rotation
[ ] Materials
[ ] Cameras
[ ] Lights
[ ] Animation
[ ] Cameras, Mesh Collisions and Gravity
[ ] Intersect Collisions - mesh
[ ] Picking Collisions
[ ] Raycasting
[ ] Sprites
[ ] Particles
[ ] Environment
[ ] Height Map
[ ] Shadows

@PirateJC I am happy to help with videos if you like.

Woot that would be so cool @Yonet! Thanks a lot

Yonet that's awesome!  We'd love to have you contribute to this!  Can you send me a PM on the forum so we can chat more about it?  https://forum.babylonjs.com/u/PirateJC
Thanks!

Moved this issue to our documentation repo under #1955


Comedy Central (cc.com) fails with error "'NoneType' object has no attribute 'strip'"

Downloading videos from Comedy Central (cc.com) used to work but no longer works. Here's what happens now:
$ ./youtube-dl 'http://www.cc.com/shows/broad-city/full-episodes'
[ComedyCentral] full-episodes: Downloading webpage
[ComedyCentral] bac44717-ef70-4e2b-a5df-3eb9b51d4638: Downloading info
[ComedyCentral] f83181b9-e9fc-4e6f-8f67-9073c05b9746: Extracting information
[ComedyCentral] f83181b9-e9fc-4e6f-8f67-9073c05b9746: Downloading video urls
ERROR: 'NoneType' object has no attribute 'strip'

Already fixed.


upgrade versions in pom to match build.xml

This pull request was merged in ManifoldCF 2.22.1:
https://github.com/apache/manifoldcf/blob/trunk/CHANGES.txt#L68


Feature Request: Provision to put test files in feature folder instead of tests folder

It will be great if we can put our test files in the respective feature folder instead of putting all the test files in app/tests folder.
When we do development for web version of angular application, we put the test files in respective feature folders with .spec.ts suffix as per the angular style guide.
If we can do the same thing in angularNative app also, it will be very easy to get started and it will open a way for unit-test code sharing. As currently we have different structures for test files in nativescript and web version, we cannot share the unit-test files easily and we need to manually copy the unit-test files and change the import path of the module being tested in test file for any cross platform shared code.
I thought to run a script which copies all the .spec.ts file to tests folder from feature folders, but then it will add complexities like adding new/changed test files back to app/tests folder in watch mode.
Let me know if there is already some way and I am missing that.
Thanks.

This issue was moved to NativeScript/nativescript-cli#2095


Refactor cf marketplace command

Refactor cf marketplace command to the new style, and add integration tests. The changes are detailed in full here:
https://www.pivotaltracker.com/story/show/160686306
Thanks,
Henry && @nmaslarski

This PR has been created off the wrong commit, I'm going to close it and we will open another.


Obtain meeting ID and organizer ID

I need to obtain meeting ID and organizer ID for old meetings that happened before subscription to notifications to form an organization-level attendance report.
Is there a way or a workaround to list all organization's meetings with their organizer IDs using application permission?
P.S. I know there isn't a direct way to pull the organizer ID from Graph API directly right now.

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

ID: d22ab715-65ae-1f44-00a4-4f37c3a81d36
Version Independent ID: d22ab715-65ae-1f44-00a4-4f37c3a81d36
Content: Obtain meeting ID and organizer ID for fetching meeting transcripts - Teams
Content Source: msteams-platform/graph-api/meeting-transcripts/fetch-id.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: lajanuar

Hello @MariamKotob - Sorry for delay in response.
We are checking this with engineering team and will share you the updates.

@MariamKotob - Engineering team has informed that "The graph onlinemeetings workload does not have an API to list the onlinemeetings for the entire tenant".


Access original request object

const contextMiddleware = ((value) => (request, ctx) => {
  return {
    // ctx.req is Express.js's request object
    user: ctx.req.user
    ...ctx,
  };
})

For example, if the user wishes to use an Express.js authentication middleware that populates req.user.
It will also enable users to access it over pageContext.req.user.
It seems to be a blocker for many use cases.

There is actually a third undocumented parameter runtime for this purpose:
const contextMiddleware = ((value) => (request, ctx, runtime) => {
  if (runtime.adapter === 'express') {
    const req = runtime.req
    // ...
  }
// ...
})

It's currently undocumented because I'm still not sure what the actual user-facing API should look like for it.

Neat it also contains res :+1:


Use BufferView::seek's return value to decide to throw

I have no idea why the original author is using uint64 and switching between them... I don't think we need<PHONE_NUMBER>3709551616 of anything.
Fixes #24

@jean-noelp Can you please take a quick look at this?


quarks not compiling

Hi,
I'm having an issue with quarks compiling in atom. It's very strange. The first time I download a quark and recompile the library the folder show's up in the post window and everything works. But if I recompile the quarks disappear again. The quarks compile fine in the IDE. I had more quarks and I uninstalled them all except for dirt-samples, superdirt, and vowel.  I guess I could just manually put quarks in the extensions folder but it seems like this normal way should be working.

Something is definitely broken for you.
It seems like it is overwriting the config file.

Hmm. Working fine for me.
Did you click "recompile class library" with the green button ?
When it compiles it posts the config:
config
configPath:	/Users/crucial/.supercollider.yaml
sclang:	/Users/crucial/code/supercollider/build/Install/SuperCollider/SuperCollider.app/Contents/MacOS/sclang
sclang_conf:	~/Library/Application Support/SuperCollider/sclang_conf.yaml

You can open that sclang_conf file and you should see your changes as they happen.
open it in Atom and you can see the file change whenever you add or remove quarks.
There is a bug that the file should be clickable but because there is a space in the path it gets URL encoded.
So on the command line you can open it:
atom ~/Library/Application\ Support/SuperCollider/sclang_conf.yaml

Yes I pressed the green button. It does seem to be updating that sclang_config file as I add quarks. But when I recompile I get this:
config
scaling: /Applications/SuperCollider/SuperCollider.app/Contents/MacOS/sclang
scaling: /Applications/SuperCollider/SuperCollider.app/Contents/MacOS/sclang
It doesn’t seem like it’s finding that sclang_config file to load the quarks.
thanks
--
Tate Carson
http://www.tatecarson.com
On June 15, 2016 at 6:21:52 AM, Chris Sattinger<EMAIL_ADDRESS>wrote:
Hmm. Working fine for me.
Did you click "recompile class library" with the green button ?
When it compiles it posts the config:
config
configPath: /Users/crucial/.supercollider.yaml
sclang: /Users/crucial/code/supercollider/build/Install/SuperCollider/SuperCollider.app/Contents/MacOS/sclang
sclang_conf:    ~/Library/Application Support/SuperCollider/sclang_conf.yaml
You can open that sclang_conf file and you should see your changes as they happen.
open it in Atom and you can see the file change whenever you add or remove quarks.
There is a bug that the file should be clickable but because there is a space in the path it gets URL encoded.
So on the command line you can open it:
atom ~/Library/Application\ Support/SuperCollider/sclang_conf.yaml
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes I pressed the green button. It does seem to be updating that sclang_config file as I add quarks. But when I recompile I get this:
config
scaling: /Applications/SuperCollider/SuperCollider.app/Contents/MacOS/sclang
scaling: /Applications/SuperCollider/SuperCollider.app/Contents/MacOS/sclang
It doesn’t seem like it’s finding that sclang_config file to load the quarks.
thanks

open the IDE and open preferences and see what the sclang_config path is.
is it different than what was posted in atom ?

My sclang_config file is here: ~/Library/Application Support/SuperCollider/sclang_conf.yaml . It's set to that in the IDE and is found fine. The problem though is that atom never loads it. Only -
config
sclang: /Applications/SuperCollider/SuperCollider.app/Contents/MacOS/sclang
sclang: /Applications/SuperCollider/SuperCollider.app/Contents/MacOS/sclang
sclang_conf never shows up in the post window.

Found the problem. A new release and fix is on the way, perhaps tonight.

will do the release soon


adding dot-path resolution for data from context in DataProviderDecorator

Just realised this may have already been accepted, but i couldn't find this commit in any of the branches when I checked.


Prometheus Metrics return "server returned HTTP status 404 Not Found"

Gitea version (or commit ref): v1.14.3 or v1.13.7

Git version: 2.32.0

Operating system: Windows Server Core 2019 or Win10 local

gitea-1.14.3-windows-4.0-amd64
Docker and / or local

Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[X]  MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[X] No

Log gist (local):

2021/06/29 11:34:49 ...ters/user/profile.go:29:GetUserByName() [E] GetUserByName: user redirect does not exist [name: metrics]
/source/routers/user/profile.go:29 (0x1feba32)
/source/routers/user/profile.go:68 (0x1febcf8)
/source/modules/web/route.go:64 (0x1fa1ffb)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/vendor/github.com/go-chi/chi/mux.go:436 (0x1b4ba6a)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/modules/web/route.go:103 (0x1fa26e9)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/modules/web/route.go:103 (0x1fa26e9)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/modules/web/route.go:103 (0x1fa26e9)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/modules/web/route.go:103 (0x1fa26e9)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/vendor/github.com/go-chi/chi/middleware/get_head.go:37 (0x21d3fe1)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/modules/context/context.go:704 (0x1b6b121)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/routers/routes/base.go:94 (0x21db6ac)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/routers/routes/base.go:94 (0x21db6ac)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/modules/public/public.go:86 (0x13d800c)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/modules/public/public.go:86 (0x13d800c)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/routers/routes/base.go:199 (0x21dd317)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/vendor/gitea.com/go-chi/session/session.go:256 (0x152ed6e)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/vendor/github.com/go-chi/chi/mux.go:70 (0x1b4956a)
/source/vendor/github.com/go-chi/chi/mux.go:311 (0x1b4fa1b)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/vendor/github.com/go-chi/chi/mux.go:436 (0x1b4ba6a)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/routers/routes/web.go:107 (0x21de7a4)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/routers/routes/base.go:38 (0x21da308)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/vendor/github.com/go-chi/chi/middleware/strip.go:30 (0x21d4927)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/vendor/github.com/chi-middleware/proxy/middleware.go:37 (0x21cfe53)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/routers/routes/web.go:63 (0x21de2e7)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/source/vendor/github.com/go-chi/chi/mux.go:87 (0x1b492f0)
/source/modules/web/route.go:298 (0x1fa151a)
/source/vendor/github.com/gorilla/context/context.go:141 (0x11980fa)
/usr/local/go/src/net/http/server.go:2069 (0x7858aa)
/usr/local/go/src/net/http/server.go:2887 (0x788ee9)
/usr/local/go/src/net/http/server.go:1952 (0x78436c)
/usr/local/go/src/runtime/asm_amd64.s:1371 (0x426640)
Description
Prometheus reports status "server returned HTTP status 404 Not Found", accessing the site in a browser, same result. I first thought it might be my change from redirecting ports, because I switched to https only without a port redirect, but it won't work with redirect turned on or http only. It was working with the first 1.14.x release(s).

What URL is prometheus trying to access here?

https://server:3081/metrics
I also reverted back to the port redirection of 3080 -> 3081 and tried everything without SSL and certificates.
Same result

I can't reproduce this on an 1.14 instance.
The 404 is likely an erroneous response that should actually be a 401 User not found (for 1.15.0 this was
fixed in https://github.com/go-gitea/gitea/pull/16069/commits/89255e93f5a30b7d107b28c560b34424b6ad8002#diff-5b3f30d78073b3263d7d49f791b6f26a19f8ed007de001cbb535aba63d1ff34c)
But this user lookup shouldn't evenbe made; there's far too much middleware running on the /metrics route.
A failure condition I can think of:

maybe the /metrics handler isn't triggered at all, as it is confused with a username
there's inconsistent state in your DB, triggered by the (unrelated) middleware looking up user redirects

Not sure where to further debug this without being able to reproduce

What tables will I have to check for inconsistencys in the DB ?

@FLeven are you sure you did enable metrics?
[metrics]
ENABLED=true

@FLeven are you sure you did enable metrics in app.ini?
[metrics]
ENABLED=true

I cannot reproduce this if enabled metrics

Yes

I cleaned up my app.ini a bit and now I get a clean 404 in the logs:
`2021/07/08 10:04:59 Started GET /metrics for <IP_ADDRESS>:49872
/source/routers/routes/base.go:36 (0x31fa2ca)
/usr/local/go/src/net/http/server.go:2069 (0x17a58aa)
/source/vendor/github.com/go-chi/chi/middleware/strip.go:30 (0x31f4927)
/usr/local/go/src/net/http/server.go:2069 (0x17a58aa)
/source/vendor/github.com/chi-middleware/proxy/middleware.go:37 (0x31efe53)
/usr/local/go/src/net/http/server.go:2069 (0x17a58aa)
/source/routers/routes/web.go:63 (0x31fe2e7)
/usr/local/go/src/net/http/server.go:2069 (0x17a58aa)
/source/vendor/github.com/go-chi/chi/mux.go:87 (0x2b692f0)
/source/modules/web/route.go:298 (0x2fc151a)
/source/vendor/github.com/gorilla/context/context.go:141 (0x21b80fa)
/usr/local/go/src/net/http/server.go:2069 (0x17a58aa)
/usr/local/go/src/net/http/server.go:2887 (0x17a8ee9)
/usr/local/go/src/net/http/server.go:1952 (0x17a436c)
/usr/local/go/src/runtime/asm_amd64.s:1371 (0x1446640)
2021/07/08 10:04:59 Completed GET /metrics 404 Not Found in 10.0839ms
/source/routers/routes/base.go:45 (0x31fa705)
/usr/local/go/src/net/http/server.go:2069 (0x17a58aa)
/source/vendor/github.com/go-chi/chi/middleware/strip.go:30 (0x31f4927)
/usr/local/go/src/net/http/server.go:2069 (0x17a58aa)
/source/vendor/github.com/chi-middleware/proxy/middleware.go:37 (0x31efe53)
/usr/local/go/src/net/http/server.go:2069 (0x17a58aa)
/source/routers/routes/web.go:63 (0x31fe2e7)
/usr/local/go/src/net/http/server.go:2069 (0x17a58aa)
/source/vendor/github.com/go-chi/chi/mux.go:87 (0x2b692f0)
/source/modules/web/route.go:298 (0x2fc151a)
/source/vendor/github.com/gorilla/context/context.go:141 (0x21b80fa)
/usr/local/go/src/net/http/server.go:2069 (0x17a58aa)
/usr/local/go/src/net/http/server.go:2887 (0x17a8ee9)
/usr/local/go/src/net/http/server.go:1952 (0x17a436c)
/usr/local/go/src/runtime/asm_amd64.s:1371 (0x1446640)
`

Ok, it was my fault, [metrics] was un-commented, in the correct app.ini, sry. Still I don't understand why the log had different error messages, the last one was the definitiv hint, that it has to be me, who caused the error. thx for looking into it anyway.


React testing library

Description
Adds React Testing Library to Testing course
HOL' UP

[x] Still needs a quiz, see issue https://github.com/wayfair/awesome-learning/issues/162
[ ] Should be merged after: https://github.com/wayfair/awesome-learning-exercises/pull/40

OK, I think I addressed all comments, ready for another look!


Add option to register alias keys in atom init config

Add ability to register alias keys without having to copy the original functions.
Use case example: register W as typo fallback for w

Thanks :+1:
Would you mind adding some specs?


show columns reports error on any tables

Describe the bug
SHOW COLUMNS cannot work. I am quite shocked why our e2e tests still passed.
Error message/log
unexpected field count in "D" message

To Reproduce
The psql history:
dev=> create table if not exists t3 (v1 int, v2 int, v3 int) append only;
NOTICE:  APPEND ONLY TABLE is currently an experimental feature.
CREATE_TABLE
dev=> show columns from t3;
unexpected field count in "D" message

./risedev slt -d dev -p 4566 -u root e2e_test/ddl/show.slt somehow returns ok for the test.
[cargo-make] INFO - Execute Command: "sqllogictest" "-d" "dev" "-p" "4566" "-u" "root" "e2e_test/ddl/show.slt"
e2e_test/ddl/show.slt                                        .. [OK] in 4139 ms

Expected behavior
No response
How did you deploy RisingWave?
No response
The version of RisingWave
PostgreSQL 9.5-RisingWave-1.3.0-alpha (unknown)
31d6dc01b543631ed13888760d1fc8ee365d649a
Additional context
No response

@neverchanje I will fix it.

I am quite shocked why our e2e tests still passed.

That's because sqllogictest only checks the return rows against to the row desc in the response.


small typo and added the word (Matrix)

changed from Riot.Im to Riot.im
and I've added Matrix in round brackets (that's how it's usually done) because Riot is just one of the many clients while the protocol is Matrix https://matrix.org/clients/ - riot is just one of the many

I understand the convention here but I'm leaning towards not merging this, I want every social platform to take as little space as possible it's a bit jarring for the eyes to have (Matrix) in text break the flow of the links. But ty for splitting up your PRs much appreciated.

Not a problem. Thanks


Open in Vega-Editor not carrying over the data

Hi, I am looking at the example provided on the package site and don't seem to be able to see any data in the Vega-Editor when it is opened. Is this an issue with how data is embedded into the svelte component?
<script lang="ts">
  import type { VisualizationSpec } from "svelte-vega";
  import { VegaLite } from "svelte-vega";

  const data = {
    table: [
      { category: "A", amount: 28 },
      { category: "B", amount: 55 },
      { category: "C", amount: 43 },
      { category: "D", amount: 91 },
      { category: "E", amount: 81 },
      { category: "F", amount: 53 },
      { category: "G", amount: 19 },
      { category: "H", amount: 87 },
    ],
  };

  const spec: VisualizationSpec = {
    $schema: "https://vega.github.io/schema/vega-lite/v5.json",
    description: "A simple bar chart with embedded data.",
    data: {
      name: "table",
    },
    mark: "bar",
    encoding: {
      x: { field: "category", type: "nominal" },
      y: { field: "amount", type: "quantitative" },
    },
  }
</script>

<VegaLite {data} {spec} />

That's by design. Your spec doesn't have the data embedded but it's passed after the chart is constructed. If you want it embedded, you need to add it in the spec itself.


Dag construction v2

@etrain @shivaram Not yet ready for merging (and plenty of cleanup to be done), but feel free to start taking a look

@etrain @shivaram This is now ready for reviewing & merging

This is looking great @tomerk - thanks for the awesome work, and for going through several iterations of this - I think we're honing in on something great!
Most of my comments are stylistic at this point, but I've also been using this branch to work on pipeline-rule rewrites and found the datastructures/APIs pretty usable.
After a round of style cleanup, given the size and importance of this patch, I'm going to ask @shivaram to  review it as well!

Also - would be great to get some unit tests on this? With all the Type _ floating around, especially!

@etrain I'll go about making these stylistic changes, and I'll see what I can do about unit tests (let me know if you have any ideas).
Also, if you haven't done so yet feel free to look at my proposal for the next version of this stuff that brings it more in line with standard practices for optimizing compilers: https://gist.github.com/tomerk/2f0021d005ac3e18d535
So long as this is usable enough for you though (and we're careful about avoiding the tricky determinism bugs we previously discussed) there's no need to start on that yet.

Re: Unit tests - it probably makes sense to construct very simple pipelines with a few different shapes, and make sure that 1) they don't NPE on construction or execution, and 2) that rule transformations "work" the way you think they should.

@etrain I've now responded to your comments & added unit tests

@etrain let me know once this is good for me to take a look

@tomerk this is looking pretty good at a high level and I just had some minor comments. I think just explaining the Pipeline data structures a bit more would make this nice and clear.
@etrain One more thing we should do is to tag the previous version as 0.1 as this is a pretty big API change.

@shivaram how's this look now?

Just some minor style comments. A couple of obligatory questions

Do the tests pass ?
Have you run some of the pipelines (say Mnist / Cifar) to see if nothing becomes slower etc ?

Yup, the tests pass!
As for running the pipelines: I have run the newsgroups & Mnist pipelines to confirm they still work correctly. I haven't been measuring performance or running on large datasets, but nothing became obviously slower.

Okay @shivaram, what about now?

LGTM. Thanks @tomerk for working through this PR !


Replace enumerate with zip.

Replace enumerate with zip.
It's more idiomatic and slightly more readable.

LGTM


Wrong value for burn rate in stirling generators

Issue Description:
Burn rate shows with octatic capacitor in stirling generator as: 1,3333334x which seems incorrect.
What happens:
Burn rate is 1,3333334x in stirling generator(when using octatic capacitor) without capacitor it shows 1.0x(which seems more logical then 1.3333334x)
What you expected to happen:
It would be logical if the burn rate followed the same logic, as without capacitor, for example 1,4x or 1,3x instead of 1.3333334x
Steps to reproduce:
1.Get EnderIO 424.
2.P

lace down stirliing generator with octatic capacitor
3.Look that it shows 1,3333334x instead of round number.
...

Affected Versions (Do not use "latest"):

EnderIO:beta 424
EnderCore:beta <IP_ADDRESS>
Minecraft: 1.7.10
Forge:1614

The value is correct per se (1.333 is 4/3), but formatting… that can be improved I believe.


Strange wording on manual

Change 'PLEASE DO NOT FORGET TO REMEMBER YOUR MNEMONIC WORDS, YOU WILL NEED THEM IN NEXT STEPS.'
to
'PLEASE REMEMBER TO WRITE DOWN YOUR MNEMONIC WORDS. YOU WILL NEED THEM IN THE NEXT STEPS.'

@sabrinasadik I assume this is an older ticket belonging to farmers and not the app? And probably already resolved there? I'll close for now, please re-open with a bit more context if my assumption was wrong.


[11.x] Update app.php config stub to match default config file

I was getting errors when trying to use the new dontMergeFrameworkConfiguration() setting.
It was because when I published all the config files the stub file for app.php was missing a few array keys.
Should these stub files always match the base config files?

was getting errors when trying to use the new dontMergeFrameworkConfiguration() setting.

Shouldn't your config/app.php contain all the possible options when you wish to use dontMergeFrameworkConfiguration()?

Please create a bug report with a reproducing repository or add integration tests to verify this issue.

Shouldn't your config/app.php contain all the possible options when you wish to use dontMergeFrameworkConfiguration()?

Yes that is correct. config/app.php needs to contain all keys, but when running php artisan config:publish --all that is not what happens. The published config file comes from config-stubs/app.php which is missing some used keys.
I guess my question really is. Should the config-stubs/app.php contain all the default and required keys that config/app.php has?

@crynobone
As suggested I have created a bug report and related example repo.
https://github.com/laravel/framework/issues/51736

Seems it would make more sense to publish from config instead of config-stubs when using don't merge framework configuration.

Submit an alternative PR #51751


Allow selection of command-mode from external contextual menu.

When one invokes the contextual menu (via a right-click) on the "Terminal" application icon (either in the start menu or from the desktop or on the taskbar) a choice of command-modes should be presented.

/dup #576


Prevent scribe from creating summaries for incorrect states

The following sequence of events causes corruption when scribe is configured for an external summarizer (isExternal is set to true)

Summarizer1 sent a summary op with ref seq # 2057
Summarizer2 sent a summary op with ref seq # 2056
Server processes summary op for ref seq # 2057 and sends a summaryNack
Server processes summary op for ref seq # 2056 and sends a summayAck

When a new client connects and downloads the summary for ref seq # 2056, the data in the summary has seq#2057 (it should be seq#2056).
This happens because:

Scribe does not revert the state for summary failures when an external summarizer is enabled.
Scribe does not have any checks regarding the summary ref sequence number. Scribe is very good at ensuring it processes ops in order, but it has no such checks for the summary ref seq num vs scribe's protocol state.

This PR adds a check for number 2 - it will ensure that scribe will not make summaries when the summarize op ref sequence number is behind its current state. Scribe will simply ignore summary ops for this case. We cannot make it nack for this case due to the potential of kafka message replays.

We cannot make it nack for this case due to the potential of kafka message replays.

If the summary op is just ignored, the client will wait for an ack for a long time (10 mins by default) before submitting another summary which can create other problems right.

Right. It would cause delays for summaries, but at least it won't cause corruption.
I also assume the client / document is already having a problem if this occurs because it implies there's more than 1 summarizer client around the same time.

I also assume the client / document is already having a problem if this occurs because it implies there's more than 1 summarizer client around the same time.

That's not entirely true. We attempt to have 1 summarizer, but this is probabilistic system - client may believe it has connection to PUSH when PUSH believes it disconnected (and communicated it to other clients). 2 summarizers will happen in such case.
What ensures correctness is CAS-like semantics of summary process, i.e. service ensuring there is single commit chain of summaries no matter how many summarizes there are.

We cannot make it nack for this case due to the potential of kafka message replays.

I'm curious how it works with the other branches (including successful flow that results in summary committed and summary ack being sent). Or failure to commit a summary due to network issue (communicating to SPO). I.e. why this case is unique?

We cannot make it nack for this case due to the potential of kafka message replays.

I'm curious how it works with the other branches (including successful flow that results in summary committed and summary ack being sent). Or failure to commit a summary due to network issue (communicating to SPO). I.e. why this case is unique?

The case isn't unique in the sense that there are 4 other replay related checks. And those checks also ignore the message/operation when triggered.

https://github.com/microsoft/FluidFramework/blob/bf6f99297564215009c160ab1e678f1b5c9df5b0/server/routerlicious/packages/lambdas/src/scribe/lambda.ts#L109
https://github.com/microsoft/FluidFramework/blob/bf6f99297564215009c160ab1e678f1b5c9df5b0/server/routerlicious/packages/lambdas/src/scribe/lambda.ts#L149
https://github.com/microsoft/FluidFramework/blob/bf6f99297564215009c160ab1e678f1b5c9df5b0/server/routerlicious/packages/lambdas/src/scribe/lambda.ts#L157
https://github.com/microsoft/FluidFramework/blob/bf6f99297564215009c160ab1e678f1b5c9df5b0/server/routerlicious/packages/lambdas/src/scribe/lambda.ts#L214

--

Handles the case were the scribe redis checkpoint worked but the corresponding kafka offset was not committed
A combination of 1 & 3
For PUSH - Handles the case where SPO's latest summary is newer than the op being processed. This is because PUSH loads/computes scribe state from SPO's latest summary on startup.
A combination of 1 & 3 - Looks like it would be hard to trigger this one on its own for PUSH. Maybe it triggers a bit differently for FRS.

Successful and most unsuccessful summaries won't be replayed due to those.
The worst possible case would be when all of the following occurs:

Summaries are unsuccessful due to SPO rejecting them, so summaryNacks are created
Scribe can't make Redis checkpoints (maybe Redis is down)
Scribe can't write Kafka offsets for the partition (maybe Kafka is having timeout issues but still allows messages to be consumed)
Scribe is restarted.
I think summaries would be replayed if all that happens and it could end up making summaryAck's for those nack'd summaries.

Hey @GaryWilber, regarding this:

Scribe does not revert the state for summary failures when an external summarizer is enabled.
Any reason why we do not make Scribe take care of that instead of addressing reason 2?

Also, regarding:

Scribe will simply ignore summary ops for this case. We cannot make it nack for this case due to the potential of kafka message replays.
I don't fully understand this kafka message replays. What would be the scenario here?

Hey @GaryWilber, regarding this:

Scribe does not revert the state for summary failures when an external summarizer is enabled.

Any reason why we do not make Scribe take care of that instead of addressing reason 2?
Also, regarding:

Scribe will simply ignore summary ops for this case. We cannot make it nack for this case due to the potential of kafka message replays.

I don't fully understand this kafka message replays. What would be the scenario here?

@hedasilv In order to solve it, we would need to update scribe to keep track of the state for every summarize op it sees. This is the state I'm referring to. It's stored as a local variable for non-external summarizers and that works fine for that case. However when external summarizers are in play, it will need to continuously track all the 'ongoing' summaries in the system. Scribe would need to start storing those in its checkpoint state so it can correctly revert when summaryNack's arrive. It would also need cleanup logic to ensure the amount of state it's tracking doesn't get too big, as well as logic to remove old entries from the state in the case where a summaryNack never appears.
I don't think it's worth investing time into solving it because PUSH is the only service running the summarizer logic externally and PUSH is on track to deprecate / stop running the scribe lambda.
"Replays" refers to the case where the lambda is restarted before it has a chance to checkpoint its state. That means it will start up at an older state than it was at before and it will consume the messages that it already processed. We don't want multiple summaryAcks/nacks for the same summarize op, so we need to handle cases like that.

If the summary op is just ignored, the client will wait for an ack for a long time (10 mins by default) before submitting another summary which can create other problems right.

Hey @GaryWilber, regarding this:

Scribe does not revert the state for summary failures when an external summarizer is enabled.

Any reason why we do not make Scribe take care of that instead of addressing reason 2?
Also, regarding:

Scribe will simply ignore summary ops for this case. We cannot make it nack for this case due to the potential of kafka message replays.

I don't fully understand this kafka message replays. What would be the scenario here?

@hedasilv In order to solve it, we would need to update scribe to keep track of the state for every summarize op it sees. This is the state I'm referring to. It's stored as a local variable for non-external summarizers and that works fine for that case. However when external summarizers are in play, it will need to continuously track all the 'ongoing' summaries in the system. Scribe would need to start storing those in its checkpoint state so it can correctly revert when summaryNack's arrive. It would also need cleanup logic to ensure the amount of state it's tracking doesn't get too big, as well as logic to remove old entries from the state in the case where a summaryNack never appears. I don't think it's worth investing time into solving it because PUSH is the only service running the summarizer logic externally and PUSH is on track to deprecate / stop running the scribe lambda.
"Replays" refers to the case where the lambda is restarted before it has a chance to checkpoint its state. That means it will start up at an older state than it was at before and it will consume the messages that it already processed. We don't want multiple summaryAcks/nacks for the same summarize op, so we need to handle cases like that.

@GaryWilber , I see, thanks for the detailed information! Regarding this part:

We don't want multiple summaryAcks/nacks for the same summarize op, so we need to handle cases like that.

I was wondering what is the trade-off here. Navin said earlier:

If the summary op is just ignored, the client will wait for an ack for a long time (10 mins by default) before submitting another summary which can create other problems right.

Is making client wait for ack for a long time (10 mins) better than submitting 2 nacks for the same op? I am probably missing something here, but what would be the client behavior upon receiving a repeated SummaryNack? Since the SummaryNack includes the original summarize op sequence number, wouldn't the client just ignore that repeated SummaryNack?
Also, since now we would skip all summarize ops on the following condition:
if (value.operation.referenceSequenceNumber >= this.protocolHandler.sequenceNumber) {

besides the scenarios described in the PR, is there any other scenario, especially for non-external SummaryWriter, where such condition may interfere negatively?
Other than those questions, the PR looks good to me and I will approve soon 😄
@pradeepvairamani to also take a look at this PR since he was also interested in this topic.

@hedasilv

Is making client wait for ack for a long time (10 mins) better than submitting 2 nacks for the same op?
I'm not sure how the client handles multiple acks / multiple nacks / an ack & nack for the same summary ref sequence number. I hope they would work correctly.

Scribe tries to never allow multiple summaryAck/Nacks for the same summary (hence all these replay checks). So we should continue keeping it that way for now to be safe. I'm not sure how the client would handle all the different cases, which include [ multiple acks / multiple nacks / an ack then nack / ack then nack ] for a given summary.

besides the scenarios described in the PR, is there any other scenario, especially for non-external SummaryWriter, where such condition may interfere negatively?

I can't think of another scenario. I could add an isExternal summarizer check to the if statement so it would only affect PUSH if needed.

@hedasilv

Is making client wait for ack for a long time (10 mins) better than submitting 2 nacks for the same op?
I'm not sure how the client handles multiple acks / multiple nacks / an ack & nack for the same summary ref sequence number. I hope they would work correctly.

Scribe tries to never allow multiple summaryAck/Nacks for the same summary (hence all these replay checks). So we should continue keeping it that way for now to be safe. I'm not sure how the client would handle all the different cases, which include [ multiple acks / multiple nacks / an ack then nack / ack then nack ] for a given summary.

besides the scenarios described in the PR, is there any other scenario, especially for non-external SummaryWriter, where such condition may interfere negatively?

I can't think of another scenario. I could add an isExternal summarizer check to the if statement so it would only affect PUSH if needed.

Thanks for the details, @GaryWilber!

I could add an isExternal summarizer check to the if statement so it would only affect PUSH if needed.

I think that might be a good idea - @pradeepvairamani, what do you think?

@hedasilv

Is making client wait for ack for a long time (10 mins) better than submitting 2 nacks for the same op?
I'm not sure how the client handles multiple acks / multiple nacks / an ack & nack for the same summary ref sequence number. I hope they would work correctly.

Scribe tries to never allow multiple summaryAck/Nacks for the same summary (hence all these replay checks). So we should continue keeping it that way for now to be safe. I'm not sure how the client would handle all the different cases, which include [ multiple acks / multiple nacks / an ack then nack / ack then nack ] for a given summary.

besides the scenarios described in the PR, is there any other scenario, especially for non-external SummaryWriter, where such condition may interfere negatively?

I can't think of another scenario. I could add an isExternal summarizer check to the if statement so it would only affect PUSH if needed.

Thanks for the details, @GaryWilber!

I could add an isExternal summarizer check to the if statement so it would only affect PUSH if needed.

I think that might be a good idea - @pradeepvairamani, what do you think?

It will definitely be the safer option. I don't have a strong opinion on this though.


feat: disabling the toast messages on view mode without debug flag

Description
When the application is in published mode, this Pr offers the ability to disable the error toast. Only the toasts that the user initiated are displayed; all others are disabled and messages are logged in the console.
When the view mode url has debug=true as one of the query params, then all the error toasts are visible.
PR fixes following issue(s)
Fixes #23605
Fixes #23603
Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Chore (housekeeping or task changes that don't impact user perception)
This change requires a documentation update

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true env=release

/build-deploy-preview skip-tests=true env=release

/ok-to-test sha=0af4b61

/ok-to-test sha=8269163

/ok-to-test sha=45f3ee7

/ok-to-test sha=32ca05a

/ok-to-test sha=b2deed8

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

/ok-to-test sha=46af7cc


Routing on public-bridge with tailscale

I'm not sure if that is a bug or just missing some manual configuration. But let me explain the setup first.
I have setup two pot nodes each running a list of pots over public-bridge. There is no specific pf rules except the nat provided through pot. One node uses <IP_ADDRESS>/16 and the other one <IP_ADDRESS>/16. Pots on the node can talk to each other as well all is accessible through host as expected.
I've then setup tailscale on both nodes with route advertising. I can access pots <IP_ADDRESS>/16 from host <IP_ADDRESS> and vice versa since routing table is updated through tailscale. The problem I'm heaving is that the same traffic is not working within the pot. I've tested with tcpdump on the destination pot and I can see traffic comes to it and also returns but then the packet is lost somewhere going back to the source pot and I can't find what could be missing.
Route table on <IP_ADDRESS> host
<IP_ADDRESS>/16      link#6             U       bridge0
<IP_ADDRESS>         link#2             UHS         lo0
<IP_ADDRESS>/16      link#3             US     tailscal

Route table on <IP_ADDRESS> host
<IP_ADDRESS>/16      link#6             U       bridge0
<IP_ADDRESS>         link#2             UHS         lo0
<IP_ADDRESS>/16      link#3             US     tailscal

pf anchor shows this
nat on vtnet0 inet from <IP_ADDRESS>/16 to any -> (vtnet0:0)

and
nat on vtnet0 inet from <IP_ADDRESS>/16 to any -> (vtnet0:0)

The confusing part to me is the following. The output of tcpdump on the destination node while running ping to <IP_ADDRESS> from source node.
10:47:45.654394 IP IP_FROM_TAILSCALE > <IP_ADDRESS>: ICMP echo request, id 55577, seq 6, length 64
10:47:45.668983 IP <IP_ADDRESS> > IP_FROM_TAILSCALE: ICMP echo reply, id 55577, seq 6, length 64

But when testing from within pot
10:48:07.713430 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 60185, seq 1, length 64
10:48:08.763099 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 60185, seq 2, length 64
10:48:09.778257 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP echo request, id 60185, seq 3, length 64

Any help would be more than appreciated.

Hi @urosgruber,
It would help to see all configuration files involved (including routing tables inside the network and arp tables). You also want to make sure to disable IP redirect on all nodes/pots involved to reduce the possibility of creating asymmetry by accident. (sysctl net.inet.ip.redirect=0; sysctl net.inet6.ip6.redirect=0 - also inside pots).
Assuming you're using VNET pots (where each jail has its own routing table) you have a general problem, which is that everything is directly connected.
I would recommend to change your setup to make use of a transfer network and route over it by IP (don't know much about tailscale and if that is possible).
So basically, I would use a setup like this:
Node A                                            Node B
<IP_ADDRESS>/30 ---------   <IP_ADDRESS>/30

And then create routes to each others pot networks:
Node A
route add -net <IP_ADDRESS>/16 <IP_ADDRESS>

Node B
route add -net <IP_ADDRESS>/16 <IP_ADDRESS>

This way, anything around vnet routing tables and arp tables etc. stays very easy to understand and control and traffic coming from a pot should flow correctly using its default route.
(all of this without ever having used tailscale and without knowing the content of your config files).
I'll remove the bug tag, as this isn't pot specific, but more of a general FreeBSD/networking question.

Hi @urosgruber,
Any update/input from you on this topic? (what I wrote might have been off, since I know little about tailgate)
Cheers

Sorry. Somehow missed this comment. Let me try to expand on the info and routing itself. So what Tailscale does is it uses Wireguard VPN as a base layer and then through their tool a few nice things can be done. They have this client that you can use and connect different nodes that are not on the same network together. As far as I understand the tool creates a specific interface called tailscal and then uses routing through this interface.
For example
<IP_ADDRESS>/16      link#3             US     tailscal
100.x.x.100    link#3             UHS    tailscal
100.x.x.1       link#3             UHS    tailscal

Here you can see that other traffic to other nodes are routed through their interface as well the network that one of the node is exposing. The last is configurable. On other node I can see similar but with the network that I exposed on this initial machine.
As I explained before, I can access all physical nodes and services that are running directly on the host. The Issue becomes when I use a pot that uses vnet and have bridge interface active. Traffic toolks like it's going out without the problem but then the reply is lost.  If I use just an alias then it works fine.
I tried to debug with tcpdump but haven't found where the problem is.
Node A
Host route (<IP_ADDRESS>/16 pot network)

route show <IP_ADDRESS>
   route to: <IP_ADDRESS>
destination: <IP_ADDRESS>
       mask: <IP_ADDRESS>
        fib: 0
  interface: bridge0
      flags: <UP,DONE,PINNED>
 recvpipe  sendpipe  ssthresh  rtt,msec    mtu        weight    expire
       0         0         0         0      1500         1         0

route show <IP_ADDRESS>
   route to: <IP_ADDRESS>
destination: <IP_ADDRESS>
       mask: <IP_ADDRESS>
        fib: 0
  interface: tailscale0
      flags: <UP,DONE,STATIC>
 recvpipe  sendpipe  ssthresh  rtt,msec    mtu        weight    expire
       0         0         0         0      1280         1         0

Node B
Host route (<IP_ADDRESS>/16 pot network)

route show <IP_ADDRESS>
   route to: <IP_ADDRESS>
destination: <IP_ADDRESS>
       mask: <IP_ADDRESS>
        fib: 0
  interface: tailscale0
      flags: <UP,DONE,STATIC>
 recvpipe  sendpipe  ssthresh  rtt,msec    mtu        weight    expire
       0         0         0         0      1280         1         0

route show <IP_ADDRESS>
   route to: <IP_ADDRESS>
destination: <IP_ADDRESS>
       mask: <IP_ADDRESS>
        fib: 0
  interface: bridge0
      flags: <UP,DONE,PINNED>
 recvpipe  sendpipe  ssthresh  rtt,msec    mtu        weight    expire
       0         0         0         0      1500         1         0

That is how it looks on the hosts, If I'm within the pot on each note route goes to a default route and that is like
route show <IP_ADDRESS>
   route to: <IP_ADDRESS>
destination: default
       mask: default
    gateway: <IP_ADDRESS>
        fib: 0
  interface: epair1b
      flags: <UP,GATEWAY,DONE,STATIC>
 recvpipe  sendpipe  ssthresh  rtt,msec    mtu        weight    expire
       0         0         0         0      1500         1         0

So from routing point of view I can't see anything wrong but still it gets lost at some point since no traffic goes through.

@urosgruber Do you have a minimal setup to reproduce or alternatively a host to ssh into?

I can allow ssh access with pub key

@urosgruber Ok, so it looks like this is a shortcoming in tailscale. In order to create site-to-site VPNs, you need to disable source NAT (looking at pf states on the hosts was quite funky - you can also test this without jails by assigning a source address from the subnet on ping from the host itself as in ping -S <IP_ADDRESS> <IP_ADDRESS>). Unfortunately, the required flag (--snat-subnet-routes=false) is not supported on FreeBSD.
See https://tailscale.com/kb/1214/site-to-site:

--snat-subnet-routes=false: Disables source NAT. In normal operations, a subnet device will see the traffic originating from the subnet router. This simplifies routing, but does not allow traversing multiple networks. By disabling source NAT, the end machine sees the LAN IP address of the originating machine as the source.

And: https://github.com/tailscale/tailscale/issues/5573
In that github issue I found a hint on setting TS_DEBUG_NETSTACK_SUBNETS=0 in tailscaled's environment to disable subnet routing.

So by disabling the netstack mode via export TS_DEBUG_NETSTACK_SUBNETS=0 on FreeBSD (and thus pfSense), the administrator is now on the hook for configuring pf to properly handle subnet routing (which might involve NAT), exit node routing (which again might also involve NAT), and transcribing the filtering semantics described in the Tailscale ACLs into pf rules for that host.

I tried this by setting tailscaled_env=TS_DEBUG_NETSTACK_SUBNETS=0 in /etc/rc.conf on both sides and restarting tailscaled. I also tweaked pf.conf a bit to skip lo0 and put in a pass rule, so that connections actually create state.
With these changes, it seems to work now (jails can ping each other and I could also do a simple connection via netcat between jails on both hosts). You might need to tweak your pf.conf so it's locked down properly.
Not really a pot question though ;)
p.s. You can remove ssh access again

@grembo Thanks for solving this. I was suspecting NAT and saw this par missing on tailscaled FreeBSD implementation, but never got into and checked this as well. I'll check the states created and then apply the necessary rules to lock it down. I owe you a 🍻 for this.


Flutter no longer gives nice blue widget when using ChildViewConnections in skyshell

I/flutter : -- EXCEPTION CAUGHT BY WIDGETS LIBRARY ---------------------------------
I/flutter : Exception caught while building ChildView
I/flutter : The null object does not have a method 'getContainer'.
I/flutter : NoSuchMethodError: method not found: 'getContainer'
I/flutter : Receiver: null
I/flutter : Arguments: [Instance of 'ViewContainerProxy']
I/flutter : Stack trace:
I/flutter : #0      Object._noSuchMethod (dart:core-patch/object_patch.dart:42)
I/flutter : #1      Object.noSuchMethod (dart:core-patch/object_patch.dart:45)
I/flutter : #2      _initViewContainer (packages/flutter/src/rendering/child_view.dart:40)
I/flutter : #3      _viewContainer (packages/flutter/src/rendering/child_view.dart:45)
I/flutter : #4      _viewContainer (packages/flutter/src/rendering/child_view.dart:45)
I/flutter : #5      ChildViewConnection._addChildToViewHost (packages/flutter/src/rendering/child_view.dart:123)
I/flutter : #6      ChildViewConnection._attach (packages/flutter/src/rendering/child_view.dart:152)
I/flutter : #7      RenderChildView.attach (packages/flutter/src/rendering/child_view.dart:235)
I/flutter : #8      AbstractNode.adoptChild (packages/flutter/src/rendering/node.dart:108)
I/flutter : #9      RenderObject.adoptChild (packages/flutter/src/rendering/object.dart:847)
I/flutter : #10     RenderBox&ContainerRenderObjectMixin.insert (packages/flutter/src/rendering/object.dart:1969)

Will fix.


Remove AAD params from templates and configure in API.

Resolves #2628
What is being addressed
At present each worksapce temaplte specifies the Azure Ad app registration choices. This means that settings can be chosen that cannot be executed.
How is this addressed

Use AUTO_ settings to configure workspace auth settings
Remove references to manual setting of app reg creation in API and templates.
Pass AUTO_ variables to API in terraform
[ ] Ensure workflows work  - aad tests?

/test

/test


Transition cache treats links with anchors as a different page

If a page is loaded in the cache, and you navigate to another page, and then back to that original page (but with an anchor), it is treated as a new page and not pulled from the cache.
For example, if you're on /derp, then navigate to /herp, then navigate to /derp#anchor, a whole new page will be retrieved from the server instead of pulling from the cache (there is no instant navigation like you'd expect from the cache).

This is no longer an issue with Turbolinks 5. I'd highly recommend everyone to upgrade as it is way more polished and usable.


Globalize for multiple countries unable to achieve

Hi All,
I am using globalize.js for currency formatting. Through this link https://github.com/jquery/globalize/tree/master/examples/plain-javascript i can able to achieve USD. But i need for USD and JPY. Please any one help to solve this issue.

Closing this issue in favor of http://stackoverflow.com/questions/36788552/currency-formatting-using-globalize-js-for-different-countries instead. Thanks for filing the question there.

@rxaviers could you please expand the answer with the next steps after CLDR is in place? Thanks.


[sdl2:x64-osx] incorrect build

Describe the bug
See https://discourse.libsdl.org/t/sdl2-linking-errors-on-macos/37387
Environment

OS: [macOS Big Sur 11.0.1]
Compiler: AppleClang 12.0.5

To Reproduce
Steps to reproduce the behavior:

./vcpkg install sdl2

I think you need to use
find_package(SDL2 CONFIG REQUIRED) # provided by vcpkg
target_link_libraries(${TARGET} PRIVATE SDL2::SDL2 SDL2::SDL2-static)

instead.

@JackBoosY, thanks! Replacing ${SDL2_LIBRARIES} with SDL2::SDL2 SDL2::SDL2-static did solve the issue. Could you please explain why the original version doesn't work on macOS, but does work on Linux which also builds statically?

find_package(SDL2 REQUIRED) uses FindSDL2.cmake, not SDL2Config.cmake which generated by the source. So the behavor is not the same.
SDL2 depends on some apple frameworks in OSX, so if your project link with sdl2 library, those frameworks are also needed.
There may have some bugs in FindSDL2.cmake, which caused this issue.

find_package(SDL2 REQUIRED) uses FindSDL2.cmake, not SDL2Config.cmake which generated by the source. So the behavor is not the same.

Actually, I haven't changed that line, so FindSDL2.cmake is still being used in all cases.

Actually, I haven't changed that line, so FindSDL2.cmake is still being used in all cases.

Since sdl2 in vcpkg has vcpkg-cmake-wrapper, it will always add sdl2 targets, but you still need to add the keyword CONFIG in find_package to ensure the correctness of the contents of the targets.

I think this issue is valid and should be reopened.

Exported SDL2 config should be self contained. Missing dependencies should be patched (and upstreamed).
The wrapper should add the the missing dependencies for the Find module in CMake. (Patches could be upstreamed to Kitware.)
Linking both SDL2::SDL2 SDL2::SDL2-static smells like poor output from the vcpkg tool usage heuristics, not like the actual usage requirement.

To fully clarify - I replaced this version (doesn't work on macOS):
find_package(SDL2 REQUIRED) # provided by vcpkg
target_link_libraries(${TARGET} PRIVATE {SDL2_LIBRARIES})

with this version (does work on macOS):
find_package(SDL2 REQUIRED) # provided by vcpkg
target_link_libraries(${TARGET} PRIVATE SDL2::SDL2-static SDL2::SDL2main)

So there's no CONFIG keyword and no linking SDL2 twice.

@dg0yt Did you mean that sdl2 vcpkg-cmake-wrapper needs to force users to use CONFIG mode when using find_package and fix macros which are provided by FindSDL2.cmake?

Thanks for posting this issue. The latest usage of sdl2 is as below, it could test succeed on MACOS, so I am closing this issue for now.
    find_package(SDL2 CONFIG REQUIRED)
    target_link_libraries(main
        PRIVATE
        $<TARGET_NAME_IF_EXISTS:SDL2::SDL2main>
        $<IF:$<TARGET_EXISTS:SDL2::SDL2>,SDL2::SDL2,SDL2::SDL2-static>
    )


roles object

The user setting object contains a "roles" object. Typically, if something starts with an underscore _, it implies private or internal. However this "roles" does not, and it has something the developers need: the names of the roles, not just GUID of roles.
It will save an extra trip to server to query names based on role id.
Is this just a documentation miss, or Dynamics not ready to expose this property? Does not seems have much side effects.

@ Can you point us to the doc page that you're talking about to better understand the issue.

@Nkrb  I am referring to https://docs.microsoft.com/en-us/powerapps/developer/model-driven-apps/clientapi/reference/xrm-utility/getglobalcontext/usersettings
Basically the object populated an property with name "roles" (not "_roles"), but the documentation for object "usersettings" does not describe that property. I'd like to see it becomes a documented property because this is highly used information, without it been documented, the developers who wanted to stay on the supported/documented side have to make a fetchXml or odata query to server to retrieve those information, which is inefficient and error prone.  Hope this make sense.

@Nianwei The userSettings.roles method returns  a collection of lookup objects containing the GUID and display name of each of the security role or teams that the user is associated with. Closing the issue. Please reopen the issue if you're not happy with the information provided.


How to build a framework for iOS?

How to build a framework package that can be used on iOS, such as .Framework /.a, Is there any documentation like this for android?

I'm interested in this as well

Duplicate of #364