[Backport - 3.0.x] Modified logging to include the resolver IP address and override the …

…… (#3364)

Modified logging to include the resolver IP address and override the client address when EDNS0 client subnet extensions are enabled and a client subnet is present. Updated docs and tests. This resolves #3363.

Updated the changelog to include a note about the logging change.

(cherry picked from commit 60ca6c409ea6270fb671b5ffb0f53ff37c55a037)
Which issue is fixed by this PR? If not related to an existing issue, what does this PR do?
Fixes #3363
Which TC components are affected by this PR?

[X] Documentation
[ ] Grove
[ ] Traffic Analytics
[ ] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[X] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] Other _________

What is the best way to verify this PR? Please include manual steps or automated tests.
(If no tests are part of this PR, please provide explanation as to why no tests are included.)
Check all that apply

[X] This PR includes tests
[X] This PR includes documentation updates
[X] This PR includes an update to CHANGELOG.md
[X] This PR includes all required license headers
[ ] This PR includes a database migration (ensure that migration sequence is correct)
[ ] This PR fixes a serious security flaw. Read more: www.apache.org/security

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3347/
Test PASSed.


[Snyk] Security upgrade socket.io from 2.4.1 to 2.5.1

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

backend/package.json
backend/package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Uncaught Exception SNYK-JS-SOCKETIO-7278048
  721

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Uncaught Exception

Olá!
Obrigado por enviar o pull request. No entanto, sua descrição não atende aos critérios estabelecidos para uma boa descrição de pull request. É importante explicar claramente o motivo da mudança e incluir um passo a passo para testar o pull request na prática.
Por favor, revise sua descrição e adicione essas informações para melhorar a qualidade do seu pull request. Se precisar de ajuda ou orientação, estou à disposição.
Obrigado pela colaboração!
Esse comentário foi gerado por inteligência artificial


Change to google formatter library

Fixes #19

Awesome! Happy to adopt it in Akka once shipped :)

👍 Awesome!


Typo error in installation instructions

There seem to be significant typo errors in the installation instructions. Please assign the issue to me.

Thanks, I'm counting on you 😃


Why RedissonTransactionManager do not support Suspend Transaction?

RedissonTransactionManager  do not rewrite
protected Object doSuspend(Object transaction) throws TransactionException
method

Done


Dim Video On Some Android Devices

For some reason so far on an Android 6.0 Samsung Galaxy 6 the video's brightness is normal but on Android 6.0 for an LG-G4, Android 7.1.1 Pixel and 6P the video is dim. Both have the same code and the same video source.
It is quite confusing. I suspect it is an issue with ExoPlayer.

Please take this up with Google ExoPlayer


Add update notification instead of banner

Closes #1005
Visual demonstration

https://user-images.githubusercontent.com/19550456/195081806-639d32bf-5b20-42c7-82c0-603ec1dd149c.mov
Also took this opportunity to turn ToolpadShell into a folder

Please follow the the contributing guide on how to create great pull requests.

Looks good. Should Community be called Community edition ?


Allow passing Infinity to waitFor as timeout

Closes #3214
WIP for now as I need figure out how to write a test for this, any pointers?

WIP for now as I need figure out how to write a test for this, any pointers?

You could create a Promise.race between your waitFor and some short-lived setTimeout (wrapped in a promise). Just to ensure that the waitFor doesn't reject immediately


Pax spinal

seems like we are missing an update to parcellation-artifacts.ttl?

This needs to be made against the parcellation branch.


Exception passing Null to facade function

Reported externally:  If you pass a null value to a facade function, it will throw an exception.

#20


WIP. Using in Async should not throw on null

fixes #1231

[x] reproduce as test case with
[x] test case with structs
[ ] fix issue
[x] add new method to surface area
[ ] add RFC for the new method

08:48:43 1) Failed : FSharp-Tests-Tools+Bundle.bundle
08:48:43 Error running command 'C:\PROGRA~2\MICROS~1\Windows\v10.0A\bin\NETFX4~1.6TO\x64\peverify.exe' with args 'test-one-fsharp-module.exe /nologo' in directory 'D:\j\workspace\release_ci_pa---866fd2c3\tests\fsharp\tools\bundle'. ERRORLEVEL 1 ERRORLEVEL 1
08:48:43 at NUnitConf.checkTestResult(Result`2 result) in D:\j\workspace\release_ci_pa---866fd2c3\tests\fsharp\nunitConf.fs:line 14

meh. what's now wrong?

That indicates the code being generated in FSharp.Core doesn't pass peverify (the "bundle" test is for static linking, and then runs peverify on the overall result)

Yeah I understood that part. I just have no idea how to fix it
On Jun 1, 2016 18:13, "Don Syme"<EMAIL_ADDRESS>wrote:

That indicates the code being generated in FSharp.Core doesn't pass
peverify (the "bundle" test is for static linking, and then runs peverify
on the overall result)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/visualfsharp/pull/1233#issuecomment-223044302,
or mute the thread
https://github.com/notifications/unsubscribe/AADgNCSwYw_QnSPRqVsVA_oOnOUcxlnDks5qHa-5gaJpZM4IrQtZ
.

OK....
There is a problem here.
IsNotNull is ultimately inlined into the generic method "Using" (which is generic over the type T of the input).   This means we can't actually statically resolve whether the type parameter is a struct or not.  It also means that neither the code for IsNotNull nor its inlined version used in Using will be verifiable.
I believe the correct way to handle this is to call the existing function IntrinsicFunctions.Dispose
            /// A compiler intrinsic for the efficient compilation of sequence expressions
            [<CompilerMessage("This function is for use by compiled F# code and should not be used directly", 1204, IsHidden=true)>]
            val Dispose<'T when 'T :> System.IDisposable > : resource:'T -> unit

This function calls box x on its input and compares it to null using this IL sequence:
  IL_0000:  ldarg.0
  IL_0001:  box        !!T
  IL_0006:  brfalse.s  IL_0018

I am pretty certain that this code emit sequence is the .NET CIL recognized way of getting generic null tests which also "disappear" when used with value types.  That is, when the type variable !!T is instantiated to a value type, the .NET JIT compilers understand that this sequence is always "true", i.e. never takes the branch.  However you can check in the x86/x64 code generated if you like.
So that means you should just remove IsNotNull and call IntrinsicFunctions.Dispose.
Cheers
Don

Cool. Thanks. Will try that tomorrow.
On Jun 1, 2016 18:28, "Don Syme"<EMAIL_ADDRESS>wrote:

OK....
There is a problem here.
IsNotNull is ultimately inlined into the generic method "Using" (which is
generic over the type T of the input). This means we can't actually
statically resolve whether the type parameter is a struct or not. It
also means that neither the code for IsNotNull nor its inlined version
used in Using will be verifiable.
I believe the correct way to handle this is to call the existing function
IntrinsicFunctions.Dispose
        /// A compiler intrinsic for the efficient compilation of sequence expressions
        [<CompilerMessage("This function is for use by compiled F# code and should not be used directly", 1204, IsHidden=true)>]
        val Dispose<'T when 'T :> System.IDisposable > : resource:'T -> unit

This function calls box x on its input and compares it to null using this
IL sequence:
IL_0000:  ldarg.0
IL_0001:  box        !!T
IL_0006:  brfalse.s  IL_0018
I am pretty certain that this code emit sequence is the .NET CIL
recognized way of getting generic null tests which also "disappear" when
used with value types. That is, when the type variable !!T is
instantiated to a value type, the .NET JIT compilers understand that this
sequence is always "true", i.e. never takes the branch. However you can
check in the x86/x64 code generated if you like.
So that means you should just remove IsNotNull and call
IntrinsicFunctions.Dispose.
Cheers
Don
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/visualfsharp/pull/1233#issuecomment-223048525,
or mute the thread
https://github.com/notifications/unsubscribe/AADgNBGqs-hjCQOEAZQ1HWBTST4eowvwks5qHbMSgaJpZM4IrQtZ
.

Ok cool. This seems to work.

This is ready to merge as far as I can see


Add location overrides to smoke test matrix

We are frequently seeing capacity and reliability issues for the usgovvirginia region regarding the iothub dependency of the smoke tests. This attempts to mitigate this issue by changing our region when AzureUSGovernment is the target environment in the smoke tests.
"statusMessage": {
  "status": "Failed",
  "error": {
  "code": "500019",
  "message": "Capacity is not available, retry later. If you contact a support representative please include this correlation identifier: f6c34c3a-e94d-4bdc-8657-34711e422b87, timestamp: 2021-03-06 19:15:08Z, errorcode: IH500019."
}

/azp run net - smoke-tests

Waiting on https://github.com/Azure/azure-sdk-for-net/pull/20318 for the tests to pass.

@benbp:  I just merged #21308.   You should be good to rebase.

/azp run net - smoke-tests

Looks like the new region has capacity!

@jsquire for now :)


Update device-metadata.md

All the links on this page were broken and redirected to devcenter main page.  Fixed them

Merged changes.
FYI, Keith. When creating links, remove the /en-us part of the path. Otherwise, it forces the browser chrome into English, no matter what the locale of the customer.


Swift 3 support

Hi,
Could you please add support for swift 3 xcode 8?

@zhxnlai  Hey any plan to add swift 3 supports ?

@zhxnlai Hey)) You plan support swift 3? I look to @tomkroening fork, look like worked. And maybe you add update to cocoapod?


NetworkTransformUnreliable.isServer is null

Describe the bug
I am trying to add NetworkTransformUnreliable components to GameObjects, because it is going to be dynamic. I simply use GameObject.AddComponent<NetworkTransformUnreliable>(); and set the target. But then NullReferenceExceptions appear. (see screenshots).
[IMPORTANT] How can we reproduce the issue, step by step:
Please tell us how to reproduce your issue, STEP BY STEP, with one of our built in examples.

In the networked player prefab, create a GameObject without a NetworkTransformUnreliable
Add a script to the prefab that adds a NetworkTransformUnreliable with target of the child GameObject
Try to host or connect as a client

Expected behavior
NetworkTransformUnreliable works perfectly after being added to a GameObject
Screenshots
(!! These errors only happen in the GameObjects with runtime-generated NetworkTransformUnreliables)

(these lead to if(isServer)... in NetworkTransformUnreliable)
Desktop (please complete the following information):

OS: Windows
Build target: Android
Unity version: 2022.3.34f1
Mirror branch: 89.9.0

There is no support for adding (or removing) network components at runtime...they have to be set up at design time and not changed.

I guess I'll try to make the editor add NetworkTransformUnreliable in the prefabs automatically.. Thanks though


Permissions issue when using EntryPlaceholder?

All seems well with superusers, but regular staff users get "You do not have permission to add a plugin" when trying to add plugins to the entry placeholder, despite being given all possible permissions.

Scratch that. I failed to use PlaceholderAdminMixin in my admin class. That fixed it.


Export version

Closed as this can work
import { VERSION } from "@azure/app-configuration-provider/dist-esm/version"


Create CI/CD Pipeline that produces a binary release at the end.

I want something I can point people to for pre-compiled downloads.

I want to be able to precompile binaries for release on the following targets:

linux x86_64
linux aarch64
apple darwin/whatever it's callled arch64
apple x86_64

should be able to use https://github.com/actions-rs/cargo to perform these cross compilations.


(node:5580) UnhandledPromiseRejectionWarning: TimeoutError: Navigation Timeout Exceeded: 30000ms exceeded

So, it was working for me till I updated the script with your latest version; i updated because sometimes it worked, other times it didn't (very strange) so I tried to update; I did npm install, npm run -s build, and added --no-sandbox to arguments, just like I did the first time I installed it.
Now, it gives me this:
Launching headless Chrome to perform the OpenID Connect dance... Navigating to STS login page... We are logged in. Sorry, i mean "you". (node:5580) UnhandledPromiseRejectionWarning: TimeoutError: Navigation Timeout Exceeded: 30000ms exceeded     at C:\Users\xxx\Desktop\destreamer-dev\node_modules\puppeteer\lib\LifecycleWatcher.js:142:21   -- ASYNC --     at Frame.<anonymous> (C:\Users\xxx\Desktop\destreamer-dev\node_modules\puppeteer\lib\helper.js:111:15)     at Page.goto (C:\Users\xxx\Desktop\destreamer-dev\node_modules\puppeteer\lib\Page.js:674:49)     at Page.<anonymous> (C:\Users\xxx\Desktop\destreamer-dev\node_modules\puppeteer\lib\helper.js:112:23)     at rentVideoForLater (C:\Users\xxx\Desktop\destreamer-dev\destreamer.js:72:20) (node:5580) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag--unhandled-rejections=strict(see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1) (node:5580) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
I've also tried to reinstall everything, with the old version of the script (I had it zipped), but nothing ;(
Please help me, I'm using your script to download the videos of my class :(
PS if you need the logs you can tell me where to find them (I'm not pratical of nodejs)

I can't seem to reproduce this on either Node 8 / 10 / 12 or 13.
Does the browser window navigate to the video URL after logging in?

Same happened for me. A new video was just recorded in a MS Teams meeting. After it finished, the video was available in Stream to play in real browser, but the script failed to download it. I waited like one hour, tried it again and I could download it. Maybe in the background there is some conversion?

Does the browser window navigate to the video URL after logging in?
Yes. In this test-managed-browser, we can't watch the video, because this type of browser is not supported by Stream. However, in other videos I could see the actual video for about 0.1 seconds. In the case the error occurred, I didn't see any part of the video.

Can you navigate to this URL
https://use2-2.api.microsoftstream.com/api/videos/THE_GUID_OF_YOUR_VIDEO/?api-version=1.3-private

and check if playbackUrls being returned in the JSON response has an entry with this mime type?
"mimeType": "application/vnd.apple.mpegurl"

Does the browser window navigate to the video URL after logging in?

Yes. In this test-managed-browser, we can't watch the video, because this type of browser is not supported by Stream. However, in other videos I could see the actual video for about 0.1 seconds. In the case the error occurred, I didn't see any part of the video.

I was saying the same exact thing. When the software was working right, you could watch a preview of the video, for 1 second; but when you have that error, the preview doesn't show.
Don't know if it can help knowing this behaviour.
PS: I've tried many and many times, without doing things differently... the last time for me it worked!
But I swear, I've not done anything differently, and it was the same link

https://use2-2.api.microsoftstream.com/api/videos/THE_GUID_OF_YOUR_VIDEO/?api-version=1.3-private

That returns a 401 error.

Can you navigate to this URL
https://use2-2.api.microsoftstream.com/api/videos/THE_GUID_OF_YOUR_VIDEO/?api-version=1.3-private

and check if playbackUrls being returned in the JSON response has an entry with this mime type?
"mimeType": "application/vnd.apple.mpegurl"

Login to Microsoft Stream first in your browser of choice.

{
"error": {
"code": "InvalidUrlId",
"message": "The URL resource id is invalid."
}
}

@Dam98 You have to replace THE_GUID_OF_YOUR_VIDEO in that URL to the real value.

Same for me. For the videos that I'm not able to download it gives a 404 error

I can't seem to reproduce this on either Node 8 / 10 / 12 or 13.
Does the browser window navigate to the video URL after logging in?

Ok, I have news on the bug: I can confirm that it is totally random.
It doesn't happen always now, but just with some videos: the strange thing is that after I've get the error, if I try again with the same video another time, it will work!
But it's annoying, cause you need to login everytime, and moreover sometimes you need to try two or three more times to make it work.
However, thanks for your beautiful work, I really appreciate it and hope it will be improved by the community.

We're changing the way we fetch the Media Services video URL in this PR. There's real hope things are going to get way more stable right after.

This issue seems almost resolved on my fork thanks to a change in the way puppeteer waits for the video page to load, we'll see if it's resolved for everyone after #20 will be merged in the repo

This should be fixed now. Just reopen if you still see this problem.


Switch to Juttle errors in public APIs

Instead of throwing native JS errors, ensure we use Juttle error classes
consistently in public facing APIs.

cc: @dmajda @demmer

+1


Control not available during record creation (before first save)

Hello,
Your control works very well but I noticed that it is not available on record creation. Actually, I must save for the first time my record before have access to the control.
Is it possible to have a version that will work during record creation ?
Thank you so much in advance.
Regards,
Fred-Alain Codjia

Hi,
I will check this and will get back to you as soon as possible.
Thanks and regards,
Shivakumar Paragi
On Thu, 18 Feb 2021, 20:39 FredAlainCodjia<EMAIL_ADDRESS>wrote:

Hello,
Your control works very well but I noticed that it is not available on
record creation. Actually, I must save for the first time my record before
have access to the control.
Is it possible to have a version that will work during record creation ?
Thank you so much in advance.
Regards,
Fred-Alain Codjia
[image: Control not available]
https://user-images.githubusercontent.com/78079220/108376764-9c703600-7203-11eb-8a2b-6345d56ea2cb.png
[image: after saving]
https://user-images.githubusercontent.com/78079220/108376765-9d08cc80-7203-11eb-8b83-99d5d85a21b6.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/shivuparagi/GenericLookupPCFControl/issues/3, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ALH5WWMKLENAFTAUFFHV2O3S7UUSPANCNFSM4X2O42DA
.

Hello,
Do you have any feedback regarding my issue ?
Thank you for your answer.
Regards,
Fred-Alain Codjia

Hi,
I am working on this and will get back to you soon.
Thanks and regards,
Shivakumar Paragi

Hello, I like you control it's usefull, but sadly I couldn't make it work. It does not render. Here is the  ConfigJSON Im uploading:
I, m tryng to put it in opportunity entity, to replace parentaccount look up.
{
"name": "parentaccountid", // Name of the actual lookup field
"openSearchPanelOnKeyDown": true, // true: search will be done as and when user types. false: search will be done when user clicks search icon.
"recordsThreshHoldLimit": 5000, // Number of records to be fetched upon search
"newRecordText": "Create New Account", // Text to be shown for Create Record
"focusControl": "focusControl1", // Focus control (in case of more number of Lookup fields on same form, do increament the number)
"entitySymbol": "Account", // Symbol of the entity
"lookUpCol": {
"entity": "account", // Name of the actual lookup field
"schemaName": "ParentAccountId", // Schema name of the actual lookup field
"pageUrl": "", // Relative URL of the lookup field entity form
"primaryKey": "accountid", // Primary Key of the looked up entity
"primaryFeild": "name", // Primay Key Name       //<----my comment: Is this "feild" correctly typed? "primaryFeild", or should be "primaryField"?
// Fields by which filter needs be performed (OR condition)
"fitlerTextFields": [
{
"name": "name",
"displayText": "Currency Name"
}
],
// Fields to show in results set
"fieldsToShow": [
{
"name": "name",
"displayText": "Currency Name"
}
],
// Views. First one be default. For fetchXML, have the required FETCHXML and add  at the end of the filter which will be replaced dynamically
"views": [
{
"id": 0,
"name": "Todas las personas"
}
]
}
}

Hi,
I tested it today and it is working fine in Quick Create Form. Can you
please check again and let me know?
Thanks and Regards,
Shivakumar Paragi
On Wed, 24 Feb 2021 at 14:21, FredAlainCodjia<EMAIL_ADDRESS>wrote:

Hello,
Do you have any feedback regarding my issue ?
Thank you for your answer.
Regards,
Fred-Alain Codjia
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/shivuparagi/GenericLookupPCFControl/issues/3#issuecomment-784914050,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALH5WWNCKMPJ43ABOPZKQCLTAS4XRANCNFSM4X2O42DA
.

Hi,
Thank you for your email. May I request you to please take a sample config
and modify it accordingly? Also please note that, while pasting the config
in the ConfigJSON field, please make sure to copy the entire text as the
textbox takes a max of 100 characters. I have provided a workaround for it.
Please check it.
Also regarding, making New record optional, I am working on it and will
get back to you soon.
Thanks & Regards,
Shivakumar Paragi
On Fri, 26 Feb 2021 at 21:32, vernetmatias<EMAIL_ADDRESS>wrote:

Hello, I like you control it's usefull, but sadly I couldn't make it work.
It does not render. Here is the ConfigJSON Im uploading:
I, m tryng to put it in opportunity entity, to replace parentaccount look
up.
{
"name": "parentaccountid", // Name of the actual lookup field
"openSearchPanelOnKeyDown": true, // true: search will be done as and when
user types. false: search will be done when user clicks search icon.
"recordsThreshHoldLimit": 5000, // Number of records to be fetched upon
search
"newRecordText": "Create New Account", // Text to be shown for Create
Record
"focusControl": "focusControl1", // Focus control (in case of more number
of Lookup fields on same form, do increament the number)
"entitySymbol": "Account", // Symbol of the entity
"lookUpCol": {
"entity": "account", // Name of the actual lookup field
"schemaName": "ParentAccountId", // Schema name of the actual lookup field
"pageUrl": "", // Relative URL of the lookup field entity form
"primaryKey": "accountid", // Primary Key of the looked up entity
"primaryFeild": "name", // Primay Key Name //<----my comment: Is this
"feild" correctly typed? "primaryFeild", or should be "primaryField"?
// Fields by which filter needs be performed (OR condition)
"fitlerTextFields": [
{
"name": "name",
"displayText": "Currency Name"
}
],
// Fields to show in results set
"fieldsToShow": [
{
"name": "name",
"displayText": "Currency Name"
}
],
// Views. First one be default. For fetchXML, have the required FETCHXML
and add at the end of the filter which will be replaced dynamically
"views": [
{
"id": 0,
"name": "Todas las personas"
}
]
}
}
[image: image]
https://user-images.githubusercontent.com/36961292/109323988-d90ae580-7832-11eb-9961-66f988eb04f8.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/shivuparagi/GenericLookupPCFControl/issues/3#issuecomment-786735962,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALH5WWLMPN6QTJM527NONCLTA7A2XANCNFSM4X2O42DA
.

Hello, thanks for answering. I aware of the workaround, I did it and make
sure to paste all the code. But I tried several times but I couldn't make
it work. I don't know where I'm failing when I modify your sample code.
I have a few questions:
Whats the reference entitySymbol?
Is the tag name "primaryFeild" correct? Shouldn't it be "primaryField" in
the sample code is typed like that.
When you say "actual lookup" you referring to the lookup that I'm trying to
emulate?
"Page url" it's require? where can I get it?
"fitlerTextFields" is correct? Or shouldn't it be "filterTextFields"?
In the name of the view has to be exactly the name of the view?
The "id" of the view has to be the GUID?
thanks
El sáb., 27 feb. 2021 13:38, shivuparagi<EMAIL_ADDRESS>escribió:

Hi,
Thank you for your email. May I request you to please take a sample config
and modify it accordingly? Also please note that, while pasting the config
in the ConfigJSON field, please make sure to copy the entire text as the
textbox takes a max of 100 characters. I have provided a workaround for it.
Please check it.
Also regarding, making New record optional, I am working on it and will
get back to you soon.
Thanks & Regards,
Shivakumar Paragi
On Fri, 26 Feb 2021 at 21:32, vernetmatias<EMAIL_ADDRESS>wrote:

Hello, I like you control it's usefull, but sadly I couldn't make it
work.
It does not render. Here is the ConfigJSON Im uploading:
I, m tryng to put it in opportunity entity, to replace parentaccount look
up.
{
"name": "parentaccountid", // Name of the actual lookup field
"openSearchPanelOnKeyDown": true, // true: search will be done as and
when
user types. false: search will be done when user clicks search icon.
"recordsThreshHoldLimit": 5000, // Number of records to be fetched upon
search
"newRecordText": "Create New Account", // Text to be shown for Create
Record
"focusControl": "focusControl1", // Focus control (in case of more number
of Lookup fields on same form, do increament the number)
"entitySymbol": "Account", // Symbol of the entity
"lookUpCol": {
"entity": "account", // Name of the actual lookup field
"schemaName": "ParentAccountId", // Schema name of the actual lookup
field
"pageUrl": "", // Relative URL of the lookup field entity form
"primaryKey": "accountid", // Primary Key of the looked up entity
"primaryFeild": "name", // Primay Key Name //<----my comment: Is this
"feild" correctly typed? "primaryFeild", or should be "primaryField"?
// Fields by which filter needs be performed (OR condition)
"fitlerTextFields": [
{
"name": "name",
"displayText": "Currency Name"
}
],
// Fields to show in results set
"fieldsToShow": [
{
"name": "name",
"displayText": "Currency Name"
}
],
// Views. First one be default. For fetchXML, have the required FETCHXML
and add at the end of the filter which will be replaced dynamically
"views": [
{
"id": 0,
"name": "Todas las personas"
}
]
}
}
[image: image]
<
https://user-images.githubusercontent.com/36961292/109323988-d90ae580-7832-11eb-9961-66f988eb04f8.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/shivuparagi/GenericLookupPCFControl/issues/3#issuecomment-786735962
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ALH5WWLMPN6QTJM527NONCLTA7A2XANCNFSM4X2O42DA
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/shivuparagi/GenericLookupPCFControl/issues/3#issuecomment-787099664,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIZ7YDANPPJWE2EGBD2YDF3TBENY5ANCNFSM4X2O42DA
.

Hello I made it work!, But I have a requirement could I hide the "New"
button?
El sáb., 27 feb. 2021 13:38, shivuparagi<EMAIL_ADDRESS>escribió:

Hi,
Thank you for your email. May I request you to please take a sample config
and modify it accordingly? Also please note that, while pasting the config
in the ConfigJSON field, please make sure to copy the entire text as the
textbox takes a max of 100 characters. I have provided a workaround for it.
Please check it.
Also regarding, making New record optional, I am working on it and will
get back to you soon.
Thanks & Regards,
Shivakumar Paragi
On Fri, 26 Feb 2021 at 21:32, vernetmatias<EMAIL_ADDRESS>wrote:

Hello, I like you control it's usefull, but sadly I couldn't make it
work.
It does not render. Here is the ConfigJSON Im uploading:
I, m tryng to put it in opportunity entity, to replace parentaccount look
up.
{
"name": "parentaccountid", // Name of the actual lookup field
"openSearchPanelOnKeyDown": true, // true: search will be done as and
when
user types. false: search will be done when user clicks search icon.
"recordsThreshHoldLimit": 5000, // Number of records to be fetched upon
search
"newRecordText": "Create New Account", // Text to be shown for Create
Record
"focusControl": "focusControl1", // Focus control (in case of more number
of Lookup fields on same form, do increament the number)
"entitySymbol": "Account", // Symbol of the entity
"lookUpCol": {
"entity": "account", // Name of the actual lookup field
"schemaName": "ParentAccountId", // Schema name of the actual lookup
field
"pageUrl": "", // Relative URL of the lookup field entity form
"primaryKey": "accountid", // Primary Key of the looked up entity
"primaryFeild": "name", // Primay Key Name //<----my comment: Is this
"feild" correctly typed? "primaryFeild", or should be "primaryField"?
// Fields by which filter needs be performed (OR condition)
"fitlerTextFields": [
{
"name": "name",
"displayText": "Currency Name"
}
],
// Fields to show in results set
"fieldsToShow": [
{
"name": "name",
"displayText": "Currency Name"
}
],
// Views. First one be default. For fetchXML, have the required FETCHXML
and add at the end of the filter which will be replaced dynamically
"views": [
{
"id": 0,
"name": "Todas las personas"
}
]
}
}
[image: image]
<
https://user-images.githubusercontent.com/36961292/109323988-d90ae580-7832-11eb-9961-66f988eb04f8.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/shivuparagi/GenericLookupPCFControl/issues/3#issuecomment-786735962
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ALH5WWLMPN6QTJM527NONCLTA7A2XANCNFSM4X2O42DA
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/shivuparagi/GenericLookupPCFControl/issues/3#issuecomment-787099664,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIZ7YDANPPJWE2EGBD2YDF3TBENY5ANCNFSM4X2O42DA
.

Hello,
@vernetmatias can you create your own issue thread so it will be clearer for anyone who is searching for answers ?
This one was created for enabling the control during a record creation.
@shivuparagi I hope you are doing fine. Please tell me when you have any solution for me :)
Thank you
Regards
Fred-Alain Codjia

Ok, I will do that.
El dom, 28 feb 2021 a las 14:15, FredAlainCodjia<EMAIL_ADDRESS>escribió:

Hello,
@vernetmatias https://github.com/vernetmatias can you create your own
issue thread so it will be clearer for anyone who is searching for answers ?
This one was created for enabling the control during a record creation.
@shivuparagi https://github.com/shivuparagi I hope you are doing fine.
Please tell me when you have any solution for me :)
Thank you
Regards
Fred-Alain Codjia
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/shivuparagi/GenericLookupPCFControl/issues/3#issuecomment-787485907,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIZ7YDGDZV4TES4MB772XL3TBJ23BANCNFSM4X2O42DA
.

Hi shivuparagi ,
Where you able to fix a solution for the above? I am also facing the same issue. Your help would be appreciated.
Regards,
G Lakshmi Priyanka

Hello again Shivakumar,
Do you had the time to take a look at this issue ?
Regards,
Fred-Alain

Yes, both issues. I will post it soon.
Thank you
On Mon, 8 Mar 2021, 21:36 FredAlainCodjia<EMAIL_ADDRESS>wrote:

Hello again Shivakumar,
Do you had the time to take a look at this issue ?
Regards,
Fred-Alain
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/shivuparagi/GenericLookupPCFControl/issues/3#issuecomment-792860627,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALH5WWL7N3FD5X5OQ24VKUDTCTYYBANCNFSM4X2O42DA
.

Hi,
This is fixed now. Please try with the latest solution and associated code change.
Thanks and regards,
Shivakumar

Hello Shivakumar,
I tried the new release by downloading the solution available on the link (check the screenshot)

There is no chance on my side. Is the managed solution up to date ?
Thank you
Fred-Alain


PSA - updates and corrections to data plane to control plane numeric …

…translation

OK, I think I am done editing this now, except for handling any changes from reviewer comments.

I'd love to do this. Let's find a time off-GitHub.
-N
On Sat, Sep 29, 2018 at 8:32 AM Andy Fingerhut<EMAIL_ADDRESS>wrote:

@jafingerhut commented on this pull request.
In p4-16/psa/psa.p4
https://github.com/p4lang/p4-spec/pull/678#discussion_r221421117:

+#define PSA_PORT_ID_HEADER_TO_INT(x) ((PortId_t) (PortIdUint_t) (PortIdInHeaderUint_t) (x))
+#define PSA_MULTICAST_GROUP_HEADER_TO_INT(x) ((MulticastGroup_t) (MulticastGroupUint_t) (MulticastGroupInHeaderUint_t) (x))
+#define PSA_CLONE_SESSION_HEADER_TO_INT(x) ((CloneSessionId_t) (CloneSessionIdUint_t) (CloneSessionIdInHeaderUint_t) (x))
+#define PSA_CLASS_OF_SERVICE_HEADER_TO_INT(x) ((ClassOfService_t) (ClassOfServiceUint_t) (ClassOfServiceInHeaderUint_t) (x))
+#define PSA_PACKET_LENGTH_HEADER_TO_INT(x) ((PacketLength_t) (PacketLengthUint_t) (PacketLengthInHeaderUint_t) (x))
+#define PSA_EGRESS_INSTANCE_HEADER_TO_INT(x) ((EgressInstance_t) (EgressInstanceUint_t) (EgressInstanceInHeaderUint_t) (x))
+#define PSA_TIMESTAMP_HEADER_TO_INT(x) ((Timestamp_t) (TimestampUint_t) (TimestampInHeaderUint_t) (x))
+
+#define PSA_PORT_ID_INT_TO_HEADER(x) ((PortIdInHeader_t) ((PortIdInHeaderUint_t) ((PortIdUint_t) (x))))
+#define PSA_MULTICAST_GROUP_INT_TO_HEADER(x) ((MulticastGroupInHeader_t) (MulticastGroupInHeaderUint_t) (MulticastGroupUint_t) (x))
+#define PSA_CLONE_SESSION_INT_TO_HEADER(x) ((CloneSessionIdInHeader_t) (CloneSessionIdInHeaderUint_t) (CloneSessionIdUint_t) (x))
+#define PSA_CLASS_OF_SERVICE_INT_TO_HEADER(x) ((ClassOfServiceInHeader_t) (ClassOfServiceInHeaderUint_t) (ClassOfServiceUint_t) (x))
+#define PSA_PACKET_LENGTH_INT_TO_HEADER(x) ((PacketLengthInHeader_t) (PacketLengthInHeaderUint_t) (PacketLengthUint_t) (x))
+#define PSA_EGRESS_INSTANCE_INT_TO_HEADER(x) ((EgressInstanceInHeader_t) (EgressInstanceInHeaderUint_t) (EgressInstanceUint_t) (x))
+#define PSA_TIMESTAMP_INT_TO_HEADER(x) ((TimestampInHeader_t) (TimestampInHeaderUint_t) (TimestampUint_t) (x))
I know @jnfoster https://github.com/jnfoster believes there is a better
way of approaching this whole area, but may not have free time to work on
it. I'm not trying to twist your arm on this, Nate, if that is still the
case, but if you might be able to find 30 mins to brainstorm any ideas you
have had on the topic with Calin and I, maybe we could take the ball and
try running with it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/p4lang/p4-spec/pull/678#discussion_r221421117, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABwi0te8BimZhwSeZgnLjXw4RaSnh1rcks5ufzAJgaJpZM4W-Egc
.

@cc10512 Do we think this is ready to be merged in?

Yes, certainly.


IPA synthetic data generation for benchmarking IPA attribution accuracy

This command line tool generates synthetic source/trigger events based on real-life sample data. Generated data help us test IPA protocols to see if they are implemented correctly to generate attribution reports, as well as measure realistic benchmark.
I splitted the diff into 3 parts.

Command line structopts - 2a06f92
Sample data - 6b431ca
Data generation - a606358

@thurstonsand @akoshelev
Thanks for reviewing! As discussed in the chat, I'll move hard-coded sample data out and create a function to read the config file as input. I'll keep this PR open for now for the sake of other comments, but I'll remove this once the newer version is ready.


fix: title wrap and hamburger offset on mobile

Fix title wrap on mobile
Fix hamburger button offset on mobile

https://github.com/dendronhq/dendron/issues/2159
Video:
https://drive.google.com/file/d/1vCqYasHUxp6ZvB4aAlmq7HIvvpEJWfmz/view?usp=sharing
https://drive.google.com/file/d/1PZt3kOkrbFJIM5lsz7UjaFd8cutvzrBy/view?usp=sharing
Pull Request Checklist
If this is your first time submitting a pull request to Dendron, copy and paste the full Dendron Review Checklist into this request and check off each item as necessary.
This template contains the short checklist which is used by the Dendron core team.
Testing

[ ] Write Tests
[ ] Confirm existing tests pass
[x] Confirm manual testing
[ ] If your tests changes an existing snapshot, make sure that snapshots are updated
[ ] If you are adding a new language feature (graphically visible in VS Code/preview/publishing), make sure that it is included in the test workspace

Docs

[ ] if your change reflects documentation changes, also submit a PR to dendron-site and mention the doc PR link in your current PR

Analytics

[ ] if you are adding analytics related changes, make sure the Telemetry docs are updated

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

thanks for the issue - can you also sign our CLA to appease the CLA bot?

@kevinslin done.

can you rebase to remove merge conflict? otherwise, lgtm

@kevinslin done


[Bug - 3.35] Remote Storage Tablet Crash

http://pastebin.com/gQPs1qRQ
This is triggered by scrolling the mousewheel immediately after right clicking to open a remote storage tablet in your hotbar (Before the UI opens).
Haven't checked to see if it's present on modern versions.

This bug has been fixed recently


headscale: 0.3.4 -> 0.3.6

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/juanfont/headscale/releases.
meta.description for headscale is: "An implementation of the Tailscale coordination server"
meta.homepage for headscale is: "https://github.com/juanfont/headscale"
meta.changelog for headscale is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

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

Warning: no invocation of /nix/store/s5hfli4i8fmjxsz0djgkl2icf56djb26-headscale-0.3.6/bin/headscale had a zero exit code or showed the expected version

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

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

found 0.3.6 with grep in /nix/store/s5hfli4i8fmjxsz0djgkl2icf56djb26-headscale-0.3.6

found 0.3.6 in filename of file in /nix/store/s5hfli4i8fmjxsz0djgkl2icf56djb26-headscale-0.3.6

directory tree listing: https://gist.github.com/ebd62200e0554039d6ac0a2e81977414

du listing: https://gist.github.com/d6564b08574752991ffd876bd6231f31

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

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
headscale

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/s5hfli4i8fmjxsz0djgkl2icf56djb26-headscale-0.3.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

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

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

1 package built:

headscale

Maintainer pings
cc @NKJe @kalbasit for testing.

Result of nixpkgs-review pr 132383 run on x86_64-linux 1

1 package built:

headscale


Scanning of a QR code doesn't work anymore

I have tested this behavior on three different devices. If I click on "add Tunnel from QR-Code", a error message stating "invalid QR-Code" appears even though the camera has not yet been started.

I have tested this behavior on three different devices. If I click on "add Tunnel from QR-Code", a error message stating "invalid QR-Code" appears even though the camera has not yet been started.

Hello! What version is this happening on and on what devices? I recently changed the QR implementation in version 3.0.0 but I have not pushed this version to the stores yet.

I tested it with a Pixel 5 and two Xiaomi devices. Currently the version installed is 2.4.5.

I tested it with a Pixel 5 and two Xiaomi devices. Currently the version installed is 2.4.5.

Do you have Google Play Services disabled? The implementation of the QR code scanning on 2.4.5 relies on play services to assist with the QR code scanning. In the latest version 3.0.0, I removed this play services dependency and swapped to a fully open source solution. This new version will go live on F-Droid in the next couple days, after which I was planning to release on Google Play.

I changed my WG config and wanted to update all of my devices, but the scanning feature didn't work anymore, even though I used to set them up by the code initially. Deleting all app data and reinstalling didn't work either.
In the end I downloaded the config file and had to use my pc to transfer it.
Yes, I have GPlay on all devices and it has already worked earlier.
Interesting is the behavior on a "new" device (my tablet the app never has been installed on), there the QR module started on the first try.

That is some really bizarre behavior. Thank you for letting me know about it! I double checked and error is coming from the old implementation that is now removed so this should get resolved with the new version. I will keep the issue open for now until the new version drops and you are able to validate that the issue is resolved.


Bootstrap grid dynamic page size

Page size is not changes for grid paging . for example default page size is set 10 . and it need to change lets 25,50 or 100 runtime based on drop down change . it desnot not change .

@yogeshelite  Will add the dynamic page size support in the next release.

thanx a lot . I realy appreciate it.
On Wed, Apr 26, 2023 at 4:13 PM Vikram Reddy @.***>
wrote:

@yogeshelite https://github.com/yogeshelite Will add the dynamic page
size support in the next release.
—
Reply to this email directly, view it on GitHub
https://github.com/vikramlearning/blazorbootstrap/issues/196#issuecomment-1523206165,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD4Y4QNLXODRFTFG4YY4LF3XDD337ANCNFSM6AAAAAAXMHDJ4U
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Thanks and Regards,
Yogesh Kumar Sharma
(Sr. Developer)
Elite Web Technologies

DISCLAIMER: This e-mail and all attached and/or related communications
are strictly CONFIDENTIAL and may contain copyright and/or legally
privileged information. They are intended solely for the use of the
individual or entity to whom they are addressed. No one else may read,
print, store, copy, forward, distribute or otherwise take any action
thereon.
The use of e-mail for any unlawful purpose or for any purpose other than as
permitted, allowed and/or intended by the above entities is strictly
prohibited. If you received this communication by mistake, please notify us
immediately, delete the message from your system, and destroy any hard
copies. Any unauthorized access to, review,reproduction, dissemination,
disclosure or other unauthorized use of the information contained herein,
on any part thereof, is also strictly prohibited..


[Bug] style of perceptor tab in profile page crashes

Version
1.7.0
Current Behavior

Expected Behavior
Not as above.
Environment
- OS: macOS Monterey M1
- Browser: 105

Any additional comments?
Pending for a few days to see if it is really a GitHub style change

Anyway, we would need to modify the style to be conformed with our new features.

Me too.

Environment

- OS：Windows 11 Insider Preview
- Browser：Chrome 106


Allow _ as a wildcard character in LIKE queries

(This is transferring https://github.com/agershun/alasql/pull/378 from a PR on master to a PR on develop. Apologies for not noticing this commit flow initially).
The SQL standard uses _ as a single-character wildcard. This PR adds in support for _ as a single character by replacing it with . during Regexp construction.
Thanks to @mathiasrw for adding the extra test file for this!

:+1:


Removed duplicate fix_func for MPS

Realized that the fix_func code block is duplicated twice in generate.py. I have checked the code and confirmed that both are identical. This PR will remove the duplicate code.

Closing this and recreated another one to merge into development instead of main


It does not support DroidSansFallback.ttf?

DroidSansFallback.zip

@huangyifu
jsPDF-CustomFonts-support.zip
Sorry for late reply.
Would you like to try this? I think the ttf file you gave me was broken and I downloaded it from somewhere else.


[WFRP4e System] Malignant Influences miscast

Page 236 Core Rules:
Malignant Influences
Casting close to sources of Corruption makes controlling the Winds of Magic much more difficult. When attempting a Language (Magick) or Channelling Test in the vicinity of a Corrupting Influence (see page 182), any roll of an 8 (representing the eight-pointed symbol of Chaos) on the units die results in a Minor Miscast as the Magic goes wild. If you already have a Minor Miscast on the Test for another reason, the Miscast escalates to a Major Miscast.
The text says "unit die" but I got it when an NPC got 8 on the "tens die".From: mogrogh#4778wfrp4e: 3.5.1wfrp4e-core: 1.3.3wfrp4e-dotr: 1.0.0wfrp4e-eis: 1.2.2wfrp4e-rnhd: 1.4.1wfrp4e-starter-set: 1.4.3wfrp4e-ua1: 1.0.2

The Other effect is from the EIS Dhar miscast rules, is that what you're referring to?

Oops, hadn't read that book yet. See that now. You can close this then.


Error on accepting output of "edit"

Hello,
I encounters today the error of below when hitting c-y to accept output of edit.

E5108: Error executing lua: ...vim/lazy/ogpt.nvim/lua/ogpt/flows/actions/edits/init.lua:225: Expected Lua number
stack traceback:
[C]: in function 'nvim_buf_set_text'
...vim/lazy/ogpt.nvim/lua/ogpt/flows/actions/edits/init.lua:225: in function <...vim/lazy/ogpt.nvim/lua/ogpt/flows/actions/edits/init.lua:222>

Try pulling master and rerun, there was a huge merge from the dev branch today. Hopefully this is fixed in that merge, if not please post again

i just tried pulling the main brancn and the same error exists:

i just fixed the issue, thank you for reporting it! try pulling main branch again!

https://github.com/huynle/ogpt.nvim/pull/20

#20

Thank you !

Hi,
i found today maybe the same error of using default docstring action:


Bugfix/module resolution

So the dynamic import adjustments I made earlier today in PR #626 didn't 100% take due to an oversight of mine.
Due to the default moduleResolution setting in TypeScript, the dynamic import() statement I added to support ESM ended up being completely transpiled out of the built JavaScript file.
This module resolution update fixes that, here's the difference in the compiled code:
- const plugin = await Promise.resolve().then(() => require(require.resolve(pluginPath, { paths: [process.cwd()] })));
+ const plugin = await import(require.resolve(pluginPath, { paths: [process.cwd()] }));

Here's the TypeScript Docs Site Reference for module resolution if you're curious at all

@jmcdo29 do you think I should leave the changeset version update as patch or release this one as minor? I don't think its a major version update, though I suppose we could compare the compiled output of each of the files to be certain?

@jmcdo29 do you think I should leave the changeset version update as patch or release this one as minor? I don't think its a major version update, though I suppose we could compare the compiled output of each of the files to be certain?

I think if this is the only part of the compiled output that is modified then the patch release is fine, as it is a bugfix. Though, I suppose support of of ESM plugins probably should have been a minor? Ah! This is where it gets difficult sometimes. You know what? Just modify the changeset file to read minor instead of patch and we'll call it good


Add Zero network

Add Zero network as per https://developers.ledger.com/docs/device-app/develop/code/clone#ethereum-clones
Description
Please provide a detailed description of what was done in this PR.
(And mentioned if linked to an issue docs)
Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)
[ ] Tests
[ ] Documentation
[ ] Other (for changes that might not fit in any category)

Breaking changes
Please complete this section if any breaking changes have been made, otherwise delete it.
Additional comments
Please post additional comments in this section if you have them, otherwise delete it.

I'll create another one


Overriding styles of third party components

I understood from the following part of the docs: https://github.com/styled-components/styled-components#third-party-components that I should be able to override the styles of a non-styled 'third party' component. This does not seem to have any effect at all.
Version
1.1.3
Reproduction
https://github.com/yn5/styled-components-override
Steps to reproduce
Clone the example repo. Run npm install and npm start.
Expected Behavior
As coded in App.js I would expect the div with the text 'test' to have a red background. Or, if I misunderstood the docs it may be they should be amended.
Actual Behavior
The style applied to the 'third party' component is not applied.

Just pass className down:
import React from 'react';

export default ({ className }) => (

Test

);

@yn5 The 3rd party component should have at least be able to accept a className props. see @diegohaz 's code.

@thisguychris @diegohaz Thanks! Missed that.


Bug tracking software move to Apps segment

fixed by https://github.com/sergiomarotco/Best-practice-for-network-segmentation/commit/3b7fe16b00175eb608d272d3cdd0499332c9b743


Mention prerendering during svelte-kit preview

The documentation for svelte-kit preview does mention that it functions irrespective of your current adapter, but it took a while for it to click for me that that meant any changes made in handle wouldn't be visible when previewing. Hopefully this clarification will help someone like me in the future. :)

thanks!


Take ownership constructors for wrapper types

I can envision a factory function for CFPP types conceptually similar to CFBridgingRelease from ARC-enabled Objective-C. The purpose of which is to assign a CF object to a wrapper and release it. The most obvious benefit would be the ability to write code like CF::Array components = CFStringCreateArrayBySeparatingStrings(NULL, string, separator).
I'm planning on using CFPP in a project and I'm probably going to implement this functionality for that purpose. If you aren't opposed to the idea on principle I'd be happy to Pull Request it.

Thanks a lot for the request. : )
I'm not opposed at all to such kind of feature. In fact I recently introduced a new class to deal with those ownership issues: CF::AutoPointer.
I've checked the code in your pull request (#18), and at first sight it seems redundant.
CF::AutoPointer can be used for all standard CoreFoundation types, even those not available in CoreFoundation++, as well as types from other frameworks:
{
    CFStringRef     o1 = CFStringCreateWith...();
    CFMachPortRef   o2 = CFMachPortCreate();
    CF::AutoPointer ap( o1 );

    ap = o2; /* `o1` is released as another pointer is assigned */

} /* `o2` is released as `ap` is destroyed */

It basically works the same way as std::shared_ptr, meaning you can safely copy them, use them as functions' parameters or return types:
CFStringRef     s1  = CFStringCreateWith...();
CF::AutoPointer ap1 = s1;
CF::AutoPointer ap2 = ap1;

CF::AutoPointer Some::Func( void )
{
    return CFStringCreateWith...();
}

void Some::Func( CF::AutoPointer ap );

{
    Some::Func( CFStringCreateWith...() );
}

CF::AutoPointer also includes a generic As method, so you can get the original pointer back:
CF::AutoPointer ap = CFStringCreateWith...();
CFStringRef     s  = ap.As< CFStringRef >();

Now, as extra sugar, the As method can also be used to get CoreFoundation++ objects:
CF::String s = CF::AutoPointer( CFStringCreateWith...() ).As< CF::String >();

I'll let you check this and see if it can work for your project.
I'm leaving the issue and pull request opened for now. : )
Cheers,

Yeah that's definitely another syntactic way to accomplish a similar goal. Downside is of course that when you go to use it you have to specify the casting type. That's why I wrote the metafunc stuff, so I could have implicit casting :)

Well, in fact you don't need to specify the type if you use the GetCFObject method, as all CF++ wrappers include constructors and assignment operators for CFTypeRef and checks the type ID. So you could also use:
CF::String s = CF::AutoPointer( CFStringCreateWithCString( NULL, "hello, world", kCFStringEncodingASCII ) ).GetCFObject();

Fair enough. I still like my function better because it requires one less keyword ;)

Agree : )
CF::AutoPointer already has a typecast operator for CFTypeRef as it inherits from CF::Type.
Unfortunately, it can't be used implicitly in such a context because of the overload resolution rules, hence the need to call GetCFObject or to use a static_cast.
Best solution IMHO would be to declare constructors and assignment operators taking a const AutoPointer &.
I was thinking about a more generic approach for all wrapper classes, so the amount of boilerplate code can be reduced (eg. having a CF::Type< T > base).
This could declare all constructors for CFTypeRef, T, AutoPointer, etc.

Got a few things like that in a local branch... I'll try to clean this up and push it so you can take a look. : )

Mmm, I was just using this today and thinking “man I could really use a CF::Type” especially since my project uses a lot of custom CF-style types

Il giorno Jul 9, 2015, alle ore 11:12, Jean-David Gadina<EMAIL_ADDRESS>ha scritto:
Got a few things like that in a local branch... I'll try to clean this up and push it so you can take a look. : )
—
Reply to this email directly or view it on GitHub https://github.com/macmade/CFPP/issues/17#issuecomment-120025927.

Sure this would be great in order to support custom CF types.
And a lot easier to write wrappers for those types.
I'm going to complete the unit tests for all constructors/assignment operators, so I can safely introduce this feature, hopefully next week.

Consider using a template type
template <typename T>
class NoRetainType {
public:
   T *ref;
   NoRetainType(T *r):ref(r){}
};

used along with a macro to make code more readable
#define NO_RETAIN_CAST(x) NoRetainType<decltype(x)>(x)

and let your other constructors know that when taking a NoRetainType to not do the usual CFRetain() when assigning. (i.e. CF::String  s1 = NO_RETAIN_CAST(CFStringCreateWith...());)
This macro should always be used with Create or Copy.
Off-topic: I've also needed a C++ wrapper for some CF functionality but I'm seeing this project too late as I've already implemented my version of the wrapper. My advice is inspired from what I have in my code


Misplaced/useless PURE comments with latest vite/rollup

When building an app using fp-ts, we get a lot of warnings because of misplaced PURE comments in a few files.
The lib works, it's just distracting to see this everytime :)
../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/IOEither.js (656:0) A comment

"/*#__PURE__*/"

in "../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/IOEither.js" contains an annotation that Rollup cannot interpret due to the position of the comment. The comment will be removed to avoid issues.
../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/ReaderIO.js (23:20) A comment

"/*#__PURE__*/"

in "../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/ReaderIO.js" contains an annotation that Rollup cannot interpret due to the position of the comment. The comment will be removed to avoid issues.
../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/ReaderEither.js (624:0) A comment

"/*#__PURE__*/"

in "../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/ReaderEither.js" contains an annotation that Rollup cannot interpret due to the position of the comment. The comment will be removed to avoid issues.
../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/ReaderTask.js (28:22) A comment

"/*#__PURE__*/"

in "../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/ReaderTask.js" contains an annotation that Rollup cannot interpret due to the position of the comment. The comment will be removed to avoid issues.
../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/ReaderTaskEither.js (26:28) A comment

"/*#__PURE__*/"

in "../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/ReaderTaskEither.js" contains an annotation that Rollup cannot interpret due to the position of the comment. The comment will be removed to avoid issues.
../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/ReaderTaskEither.js (87:27) A comment

"/*#__PURE__*/"

in "../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/ReaderTaskEither.js" contains an annotation that Rollup cannot interpret due to the position of the comment. The comment will be removed to avoid issues.
../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/ReaderTaskEither.js (92:26) A comment

"/*#__PURE__*/"

in "../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/ReaderTaskEither.js" contains an annotation that Rollup cannot interpret due to the position of the comment. The comment will be removed to avoid issues.
../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/ReaderTaskEither.js (991:0) A comment

"/*#__PURE__*/"

in "../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/ReaderTaskEither.js" contains an annotation that Rollup cannot interpret due to the position of the comment. The comment will be removed to avoid issues.
../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/StateReaderTaskEither.js (773:0) A comment

"/*#__PURE__*/"

in "../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/StateReaderTaskEither.js" contains an annotation that Rollup cannot interpret due to the position of the comment. The comment will be removed to avoid issues.
../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/TaskThese.js (69:26) A comment

"/*#__PURE__*/"

in "../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/TaskThese.js" contains an annotation that Rollup cannot interpret due to the position of the comment. The comment will be removed to avoid issues.
../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/TaskEither.js (962:0) A comment

"/*#__PURE__*/"

in "../../.yarn/cache/fp-ts-npm-2.16.3-31e1c8b029-24f071a4a6.zip/node_modules/fp-ts/es6/TaskEither.js" contains an annotation that Rollup cannot interpret due to the position of the comment. The comment will be removed to avoid issues.

Should be fixed in 2.16.4


feat: SSH Tunnel functionality

Uses ssh2-promise for tunnel-functionality:
See also: ssh2-promise on github

Fixed also lint-error in not related (src/renderer/scss/themes/dark-theme.scss) - hopefully that's ok in this case. Otherwise pipelines will fail.

Tested with cases:

local mariadb (with and without ssl)
local postgres (without ssl)
ssh mariadb (without ssl) - with ssh-password | ssh-private-key
ssh postgres (without ssl - with ssh-password | ssh-private-key

Hello @digitalgopnik ,
Infinitely thank you for your contribution!
Everything seems to be working fine.

Great!


fix: resolve dependency and environment management issues

Take 2
Fixes a couple of environment/dependency issues that cropped up recently

@jsstevenson did you want to try and resolve the GH Actions before I review?

@korikuzma I'm not touching the tests until https://github.com/cancervariants/therapy-normalization/tree/mock_data is merged. Up to you if you want to wait til then, we don't per se need to merge this imminently

@korikuzma I'm not touching the tests until https://github.com/cancervariants/therapy-normalization/tree/mock_data is merged, they make me crazy and bring me stress. Up to you if you want to wait til then, we don't per se need to merge this imminently

@jsstevenson Ah okay, that's fine.


Require setting of KnownNetworks or/and KnownProxies when ForwardLimit is set to null.

Documentation for ForwardedHeadersOptions.ForwardLimit Property currently contains these commens:

Set to null to disable the limit, but this should only be done if KnownProxies or KnownNetworks are configured.

I asked for more detailed description of the security issues. However, I got the reply to my request #11949 :

Generally we don't do this for security issues because people simply don't read the docs and we'd rather everyone was as safe as possible.

So if ASP.NET Core wants to be strict and consistent, there should be check for the secure combination of ForwardedHeadersOptions settings and exception should be thrown in case the combination is not fine.

@Tratcher thoughts?

Throwing an exception here would be a breaking change. We could consider squeezing that in to preview 8. Given the security aspect here I'm not opposed to the idea (but would like to hear @Tratcher 's thoughts).
The only case I can see where unlimiting ForwardLimit while also not restricting KnownProxies and KnownNetworks would be if my app is running on an entirely trusted closed network. It still seems reasonable to require at least KnownNetworks in that case, since you should hopefully be able to specify the CIDR(s) that are valid in that scenario.
A less breaking alternative would be to log a warning (or even an error, without actually throwing something). But I think that's probably not really necessary here.

Conceptually I agree, but people end up with pretty complex configs and being overly prescriptive would likely prevent them from using this middleware. We focus on secure defaults and try to explain the implications of changing them.
Also, the security implications are only as severe as how you utilize the results. It's one thing if you're using the client IPs for telemetry, but quite another if you're trying to use them as an allow/deny list for authorization. Given there are a variety of other ways to spoof client IPs (e.g. VPNs), there are practical limits to how far you should trust them.

Triage decision: We try to have relatively secure defaults but validating every permutation isn't really feasible. Docs are pretty clear here. Closing as we don't intend to take action.


Generate QSF script paths relative to the current directory.

afu_synth_setup and rtl_src_config were generating paths relative to what
the scripts expected the Quartus working directory to be. Since the working
directory may change, instead generate paths relative to the directories of
the generated QSF scripts.

Coverage remained the same at 79.973% when pulling 14f486d23929e4289c0752a42248f8d8a8b830e0 on mcadler/qsf_rel into 1cb5b13c6bf8b69cd6345a623dc9d1dd58ff82bf on master.


chore(package.json): update version to 0.1.1

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@6090ef9). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             dev       #14   +/-   ##
=======================================
  Coverage       ?   100.00%
=======================================
  Files          ?         3
  Lines          ?       130
  Branches       ?         9
=======================================
  Hits           ?       130
  Misses         ?         0
  Partials       ?         0


[WSL2] Issues with running LXD (system containers)

Your Windows build number:  Windows 10 build 18917 (Fast ring)

What you're doing and what's happening:

I am using WSL2 with the default Ubuntu 18.04.2 image.
I want to setup LXD (system containers, unprivileged) to work in WSL2.
I run the following to start the service, then sudo lxd init to initialize LXD. I accept all the defaults.
sudo service lxd start
sudo lxd init

What's wrong / what should be happening instead:

The initialization fails with a networking error,
Error: Failed to create network 'lxdbr0': Failed to run: iptables -w -t filter -I INPUT -i lxdbr0 -p udp --dport 67 -j ACCEPT -m comment --comment generated for LXD network lxdbr0: iptables: No chain/target/match by that name.

Further errors
You can setup LXD without networking, by responding no to the networking questions when you run sudo lxd init. By doing so, sudo lxd init manages to complete. But when you actually try to start a new container, you can the following errors:

lxc mycontainer<PHONE_NUMBER>1018.500 ERROR    utils - utils.c:safe_mount:1179 - Operation not permitted - Failed to mount "sysfs" onto "/usr/lib/x86_64-linux-gnu/lxc/sys"
lxc mycontainer<PHONE_NUMBER>1018.500 ERROR    conf - conf.c:lxc_mount_auto_mounts:724 - Operation not permitted - Failed to mount "sysfs" on "/usr/lib/x86_64-linux-gnu/lxc/sys" with flags 0
lxc mycontainer<PHONE_NUMBER>1018.500 ERROR    conf - conf.c:lxc_setup:3539 - Failed to setup first automatic mounts
lxc mycontainer<PHONE_NUMBER>1018.500 ERROR    start - start.c:do_start:1263 - Failed to setup container "mycontainer"
lxc mycontainer<PHONE_NUMBER>1018.500 ERROR    sync - sync.c:__sync_wait:62 - An error occurred in another process (expected sequence number 5)
lxc mycontainer<PHONE_NUMBER>1018.500 WARN     network - network.c:lxc_delete_network_priv:2589 - Operation not permitted - Failed to remove interface "eth0" with index 7
lxc mycontainer<PHONE_NUMBER>1018.500 ERROR    lxccontainer - lxccontainer.c:wait_on_daemonized_start:842 - Received container state "ABORTING" instead of "RUNNING"
lxc mycontainer<PHONE_NUMBER>1018.501 ERROR    start - start.c:__lxc_start:1939 - Failed to spawn container "mycontainer"
lxc<PHONE_NUMBER>1018.506 WARN     commands - commands.c:lxc_cmd_rsp_recv:132 - Connection reset by peer - Failed to receive response for command "get_state"

Tips

To start over, you can sudo apt remove --purge lxd, then install LXD again with sudo apt install lxd.

References

Status of LXD in WSL2

It seems someone made it work.
https://twitter.com/10_Chrisi/status/1139619146935820288

I wrote a post on running LXD in WSL2.
Specifically, I could not get networking to work, so I used proxy devices.
The result is to get Internet Explorer to view the website from a LXD container.
https://blog.simos.info/how-to-run-lxd-containers-in-wsl2/

^--- Thanks for that 👍.
As you point out, at base, the WSL2 has some missing KCONFIGs ref #4165. Others have hit the OP fail here and here, and this Gentoo peeps appear to have worked-around with this patch.
Since you bring it up in your blog post, you can get systemd up on WSL2 if you are feeling risqué ref #994. Now that we're playing with the big boys, a lot of the Linux ecosystem was previously out of bounds in WSL1 because of "bigger problems" would really rather be living in a warm systemd home, especially if those package binaries were built on a distro that assumes systemd exists (because their local build scripts find systemd). That's not to say these packages won't run without systemd (they generally can), but the devs that actually maintain those packages more often than not have systemd on their rigs (typically Red Hat or Ubuntu) so it isn't too surprising there are going to be rough edges for us on WSL2. It isn't too hard to find these rough edges on the Interwebs, because you'll get Google hits from the Gentoo/Alpine/Void people.

I'm able to lxd init but containers silently stop.
My default profile
lxc profile show default
config: {}
description: Default LXD profile
devices:
  eth0:
    name: eth0
    nictype: bridged
    parent: lxdbr0
    type: nic
  root:
    path: /
    pool: default
    type: disk
name: default
used_by:
- /1.0/containers/c2

My lxdbr0
lxc network show lxdbr0
config:
  ipv4.address: <IP_ADDRESS>/24
  ipv4.nat: "true"
  ipv6.address: none
description: ""
name: lxdbr0
type: bridge
used_by:
- /1.0/containers/c2
managed: true
status: Created
locations:
- none

When I do lxc start c2 my container stays STOPPED
lxc list
+------+---------+------+------+------------+-----------+
| NAME |  STATE  | IPV4 | IPV6 |    TYPE    | SNAPSHOTS |
+------+---------+------+------+------------+-----------+
| c2   | STOPPED |      |      | PERSISTENT | 0         |
+------+---------+------+------+------------+-----------+

Container logs say:
lxc info c2 --show-log
Name: c2
Remote: unix://
Architecture: x86_64
Created: 2020/06/03 19:52 UTC
Status: Stopped
Type: persistent
Profiles: default

Log:

lxc c2<PHONE_NUMBER>3559.215 WARN     conf - conf.c:lxc_setup_devpts:1616 - Invalid argument - Failed to unmount old devpts instance
lxc c2<PHONE_NUMBER>3559.407 WARN     network - network.c:lxc_delete_network_priv:2589 - Operation not permitted - Failed to remove interface "eth0" with index 17


Configure the build-service gitops package

Should be merged after #87.
As part of the changes needed to allow the overriding of the Tekton bundle, the gitops package present in the build-service repository now needs to make use of a K8s client in order to load the necessary configmaps.
This PR adapts the HAS controller to comply to these new changes.

@brunoapimentel Is this PR still needed with https://github.com/redhat-appstudio/application-service/pull/96?

@brunoapimentel Is this PR still needed with #96?

This certainly will have a lot of changes, so I'm closing this PR for now.


Add an POC for ISv2

Example implementation of an instantsearch.js-like API, but using the architecture from IS-react under the hood.

thx this will be very handy for us :)


[FEAT] Add support for FCM Data messages

Is your feature request related to a problem? Please describe.
Currently, the app server only allows sending FCM Notification Messages. But FCM data messages are versatile and will be useful in a lot of scenarios, especially if using with native apps.
Describe the solution you'd like
The notification endpoint will have to be changed so as to abstract and add data messages as follows -
# Old
/projects/{projectId}/users/{subjectId}/notifications

# New
# for notificaitons
/projects/{projectId}/users/{subjectId}/messaging/notifications

# for data
/projects/{projectId}/users/{subjectId}/messaging/data

We will also need to add the abstract out the FcmMessage builder part in NotificationSchedulerService so as to easily add/replace it for data messages. We will also need a new Entity, Repo and Service for Data messages. Finally, we will need to add DataJob as a Quartz Job for scheduling.
Moreover, it would be good to abstract out the Sending of messages in NotificationSchedulerService so as to can be used to send using other types of push notification providers easily (though the structure of entities is based on firebase but should be easy to translate to other providers)
Describe alternatives you've considered
N/A
Priority
8
Difficulty
7
Additional context
Add any other context or screenshots about the feature request here.

closed by #66 and #63


Panic during startup

2023-06-12 12:23:33 goroutine 1 [running]:
2023-06-12 12:23:33 github.com/go-chi/chi/v5.(*Mux).Use(...)
2023-06-12 12:23:33     /go/pkg/mod/github.com/go-chi/chi/v5@v5.0.8/mux.go:102
2023-06-12 12:23:33 mock-payment-provider/presentation.NewPresenter({{0xc00001c026, 0x7}, {0xc00002000a, 0x4}, {0xc00001e00b, 0xf}, 0xc000022180})
2023-06-12 12:23:33     /app/presentation/presentation.go:53 +0x9a6
2023-06-12 12:23:33 main.main()
2023-06-12 12:23:33     /app/main.go:82 +0xa8d
2023-06-12 12:23:40 panic: chi: all middlewares must be defined before routes on a mux

Released https://github.com/TokoBapak/mock-payment-provider/pkgs/container/mock-payment-provider/102353199?tag=sha-6395d75


Properly display the final version of selected role permission

Is your suggestion related to a experience ? Please describe.
I have experience difficulties when checking the added role permission set at final wizard as mentioned below.

Describe the improvement
Request to improve the above display which shows the selected permissions. This not user friendly since it's difficult to identify the added set of permission and their hierarchy.
Better to display as a hierarchy.

Since the tree view is used to select permissions, the ideal way is to show a read only tree view in the summary as well.


Question: Confusion around keys/access policies

I was playing around trying to get this working tonight and was able to get it to return correct data as far as I can tell, but when I go to upload the file using the key, signature and policy that the endpoint returns, I get the error "The AWS Access Key Id you provided does not exist in our records."
I'm assuming I'm missing something in access policy or configuration for the lambda function or role but I haven't found much help in the docs. Any ideas?

@j--w I'm definitely going to look into this over the weekend. Could you provide the code you are using to do the upload on the browser side (at least the relevant portion)?

Thanks @binoculars. I've created a gist with the pertinent information, linked below. For now all I'm doing is sending an example request with postman to get the signature with the filename and size, then I'm manually copying the values from that response info into an html form on localhost to test. I post to my s3 bucket endpoint and get the error as outlined in the gist. Let me know if you need more info or if theres something obvious I've missed. Cheers. Once I figure this out I can submit a PR with a getting-started.md or something to help anyone else who may be checking this out.
https://gist.github.com/j--w/e41db4761ecc144b5366

@j--w Thanks for the gist. I did look into it this weekend and I believe I know what the issue is now. I have something similar (pre-JAWS) that I created a separate IAM user for and then gave it S3:PutObject permissions to the bucket. Apparently the API gateway invokes the lambda function with an ASIAXXXXX access key (vice AKIAXXXXX) which I believe means that it uses the additional security token.
TL;DR I think we need to pass the security token or create an IAM user with bucket permissions. I will try the former and if it works, update the module.

Ahh yes, that makes complete sense. I think creating a user with bucket permissions and passing it along makes sense, it allows security concerns to be separated.

I tried passing the session token and that's not working. I suppose the only way ahead is creating the IAM user. I'm going to have to figure out how to do this in CloudFormation that captures the Access Key ID and the Secret so that the function can read them in. Suggestions welcome.

Looks like it may be able to be done with:
http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-iam-user.html
and http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/intrinsic-function-reference-getatt.html
I'm likely overlooking something, I'll try to dig in a bit more detail.
Is the manual workaround to manually create an IAM user with the right policy and include the access key and secret in environment variables that get included as part of the AWSM module?

Yes, although you have to rename them if you're going to include them in process.env so that the STS stuff doesn't get overwritten. I prefer to keep them in a JSON file and just do var creds = require('creds.json');, where creds.json is something like:
{
	"AWS_ACCESS_KEY_ID": "AKID...",
	"AWS_SECRET_ACCESS_KEY": "secret..."
}

Right, that probably works for my purposes but having cloudformation provision it automatically would be awesome. One of my coworkers is looking into JAWS/cloudformation heavily right now so I'm going to plant the seed to keep this in mind as he's working with it :smile: Paging @dijitalmunky

I'm using this for testing the front end: https://gist.github.com/binoculars/e8b68e6b4e1c2e5f42ba


There is an (alpha but) official ISO!

here:
Lubuntu Next 17.10 (Artful Aardvark) Daily Build
It boots (on both my 2 laptop, for example) and finally has NM so you can connect to your wifi... It definitively works and it's also clean: manifest with shipped packages attached. Now you can built your iso over this or simply have an alternative. Thank you anyway.
artful-desktop-amd64.manifest.txt

Yeah, I already pointed to that alpha in another issue, I guess is time lo leave this project now that Lubuntu is finally switching to LxQt :)

Hello,
I would like to congratulate you for the excellent work done with UbuntuQt.
The distribution is excellent, the combination LxQt + Numix was great.
And if possible continue with the project, please do not end with UbuntuQt.
Lubuntu will come with the LxQt, but it should only arrive in the Lubuntu 18.04, Lubuntu 17.10 is still in testing.
Thank you very much in advance.
Greetings.


feat: Params for middleware

Fix https://github.com/denoland/fresh/issues/903
This PR introduce middlewareParams in the MiddlewareHandlerContext.  Each middleware have access to the params that are upstream of its definition.
For example:
With a route '/api/[id1]/[id2]/foo'
and a middleware located at '/api/[id1]/_middleware',
middlewareParams will only have a property of 'id1'. 'id2 will' be undefined
Why not all the params of the route ?
For a middleware to access the params that are downstream from its level would require, I think, to duplicate in fresh the logic that is currently in the Rutt router. This would introduce a speed decrease. One solution could be to integrate the Rutt router in fresh, although I did not want to tackle this.
While this implementation has a limitation, I think it covers most use cases and does not add any significant overhead

Note the linter is failing not because of this PR but because of a recent change in deno affecting the whole project. There a 16 instances of Deno.run being deprecated, which should probably be fixed separately

@marvinhagemeister, don't forget that this is no longer necessary due to my 1314.

Closing in favour of #1314


UWP: Un-pausing media makes it start from the beginning

If a song is paused, and you un-pause it by calling MediaManager.PlayPause(), the song starts over from the beginning instead of continuing where it left off.

Issue #89 also exists on UWP.

Still have the same issue happened using
PlaybackController.SeekTo(StreamingPosition)
MediaManager.PlaybackController.Play()


Flakey integration test

We've had some transient failures of the integration_tests::e2e_tests::test_bindle_static_assets, e.g.:
https://github.com/fermyon/spin/runs/7900758561
https://github.com/fermyon/spin/runs/7925489356
https://github.com/fermyon/spin/runs/7925553956
thread 'integration_tests::e2e_tests::test_bindle_static_assets' panicked at 'Ran out of retries waiting for ./target/debug/spin to start on URL <IP_ADDRESS>:49342', tests/integration.rs:794:17

Given that these are transient and all appeared on the macos build (which may be the slowest hardware), I suspect that the retry/timeout config is too low.

Thanks! I'll assume that will fix this.


How to infer images on multi gpu?

Hello, I want to infer an amount of images and get the last embedding of each image.
Now I use inference_on_images  function like:
args = {"num_workers": 0, "batch_size": 8}
features_queries = inference_on_images(extractor, paths=queries, transform=transform, **args)

But it seems to run only on cpu. I modify oml's code in ckpt = torch.load(weights, map_location="cpu") here to ckpt = torch.load(weights, map_location="cuda:0"})
Then I run inference_on_images, having two questions:

But the Memory-Usage on GPU does not increase, and the GPU-Util is still zero. Why?
So how to successfully infer images on gpu and multi gpu to get the last embedding of each image?

@PapaMadeleine2022 Hey, you can do it even more simpler with validate_cub.py. Just go to the metric callback and take the accumulator from that, it stores all the available information. It would be something like metric_callback.metric.acc.storage.
PS. Ideally, we need a dedicated script for inference.

For question 1, can you give some advises ?

Move your model to gpu
model.to(...)

@PapaMadeleine2022 Hey, you can do it even more simpler with validate_cub.py. Just go to the metric callback and take the accumulator from that, it stores all the available information. It would be something like metric_callback.metric.acc.storage.
PS. Ideally, we need a dedicated script for inference.

Still do not understand how to do it. Can you give some examples?

@PapaMadeleine2022 , For example, in this notebook:
https://github.com/OML-Team/open-metric-learning/blob/main/pipelines/features_extraction/visualization.ipynb
We did validation, then we took the callback. See clb_metric.metric.
If you go to clb_metric.metric.acc.storage you access all the data that was accumulated during the validation.

accumulated

If I can only access all the data that was accumulated during the validation.it seems not to fit my needs. Because my val data is hug, such that memory will not be able to store all the images.

it doesn't accumulate images, only embeddings, paths, bboxes, labels...

embeddings
Sorry, I had a wrong expression. I have fixed my above words.

You can easily calculate the required space, usually, embeddings are light enough.
If you cannot put them into the memory, you can dump them to a file or/and decrease the embedding size and retrain the model or/and use PCA (implemented by your own).


Upstream changes for allocation

Describe the bug
Goal: reduce changes in common files.
Upstream changes made in lib/Analysis/Allocation.cpp and include/triton/Analysis/Allocation.h: diff_allocation.txt
Environment details
PVC

I think it should actually be the other way around: We should adhere to the content of those files. I'll do that.


ASB-430: Update pins-components package

:tada: This PR is included in version 2.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Does GORM have a way to do a migration dry run / check if DB needs any changes?

Does GORM have a way to do a migration dry run / check if DB needs any changes?
Should be answered on migration docs page
Expected answer
I would like to be able to run a dry run to see if the database needs any migration. Something like:
func (m Migrator) NeedsMigration(dst ...interface{}) (bool, error)

For example, to check the whole DB:
m := db.Migrator()
needsMigration, _ := m.NeedsMigration()

And maybe to check a single table:
needsMigration, _ := m.NeedsMigration(&models.User{})

I looked into altering the code a bit this morning, but thought I should ask to see if anyone else has encountered this issue.

AutoMigrate will migrate if necessary, but there is not dry run mode.

@jinzhu there should be IsMigrationRequired() or something. Just to make sure it won't AutoMigrate something in production and let developer know about that migration is required. May be even print queued SQL commands.
I plan to create own Dialector, to provide custom Migrator, that will fail on any modification function. Then call AutoMigrate and catch any thrown panic().
All migrations in production servers should be explicit and launched manually with ./service migrate.

Minimal working sample. https://gist.github.com/molind/a67100448b886b7257e30799e06a0718

Minimal working sample. https://gist.github.com/molind/a67100448b886b7257e30799e06a0718

I think it is nice to translate the output to SQL format.

@defool SQL generation is DB dependent. May be there is a simple solution, but I don't know how to do it. You may try to do it yourself.

@defool  I write an example to generate mysql sql with gorm, https://github.com/PalanQu/easycoding#topic1-database-migrate


Release v5.4.1

To install this version with npm, run npm install<EMAIL_ADDRESS>You can also find more information about how to stay up to date in our documentation.
Recommended changes
Update Breadcrumbs to use nav and aria-label
We've made changes to the Breadcrumbs component to improve how it appears to screen readers.
We've changed the wrapping element to use the nav tag to expose it as a navigational landmark, and added an aria-label attribute to differentiate it as breadcrumb navigation.
This change was introduced in pull request #4995: Update Breadcrumb component to improve screen reader accessibility.
Fixes
We've made fixes to GOV.UK Frontend in the following pull requests:

#5114: Fix divider width for small checkboxes – thanks to @colinrotherham
#5043: Refactor the accordion JavaScript
#5044: Remove session storage checks from accordion JavaScript
#5060: Reintroduce additional bottom margin to Error Summary content
#5070: Fix alignment of content in conditional checkboxes and radio buttons
#5090: Remove redundant tag CSS from phase banner

Fonts diff likely linked to this update of Gulp. Percy shows no diff so all good! ⛵


mutable (and why to avoid it)

https://isocpp.org/blog/2012/12/you-dont-know-const-and-mutable-herb-sutter

https://eel.is/c++draft/res.on.data.races#3

Good link for example: https://compiler-explorer.com/z/79se63ah9

Coming in Ep439

While the recording of Herb Sutter's talk on Channel 9 doesn't seem to be available officially anymore, the Internet Archive Wayback Machine luckily has a snapshot with the video in it: https://web.archive.org/web/20170119232617/https://channel9.msdn.com/posts/C-and-Beyond-2012-Herb-Sutter-You-dont-know-blank-and-blank
(please excuse me if this is not the right place for this link suggestion, I thought here might be a better place than in the YT comments on the episode though)

(please excuse me if this is not the right place for this link suggestion, I thought here might be a better place than in the YT comments on the episode though)

This is perfect, thank you.


⚠ Minecraft has degraded performance

In 830c03d, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1215 ms

Resolved: Minecraft performance has improved in 70db4fc.


refactor: organize example apps infra into dirs

Issue or need
Example apps

Proposed changes

Quick reminders

🤝 I will follow Code of Conduct
✅ No existing pull request already does almost same changes
👁 Contributing docs are something I've taken a look at
📝 Commit messages convention has been followed
💬 TSDoc comments have been added or updated indicating API visibility if API surface has changed.
🧪 Tests have been added if needed. For instance, if adding new features or fixing a bug. Or removed if removing features.
⚙ API Report has been updated if API surface is altered.

#986  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on  Graphite


client side no longer works

i tried change true both lethal config and normal config file but it doesnt work. Its only working when i am the host

Pls answer it

Sorry, but you didn't even tell me what's wrong. It sounds like you're trying to set a config option to true, but it's not working? What config option isn't working?
Or does the whole mod not work when you're not the host?

Like i said its working when i am the host but when i joined public lobby its not
And i enabled true both config file and lethal config then i restarted but it still not working

Sorry, I thought this was the TooManyEmotes github. I understand what you're saying. Yes, I've been told that this is currently not working. I will be trying to look into this as soon as I can.

Thanks and Which version do you recommend?

If you're wanting to use this mod with other people who do not have it, then I believe the latest version before 2.0.0 (I can't remember the last 1.x.x version right now) would be your best bet, at least until I can fix this soon.

You are number 1


Updated CODEOWNERS to be in sync with keptn/community

@jetzlstorfer @johannes-b @warber I have updated CODEOWNERS as agreed on.
Please note it is possible to reference a team (e.g., @keptn/maintainers )

spellcheck issues should be fixed.


Firefox 39

For various reasons, I've stayed with Firefox 39. For several years, this has been the combination that's permitted me to have stable Flash playback on YouTube. I've kept that going, because Firefox 40 no longer allowed Flash to be stable enough. wrt other functionality, Firefox 43.0 is the last one to support cookie prompting.
I haven't buried the idea of Flash yet, but am considering YT2P.
The issue is, that context menu items do not work in version 2.0.1. When adding the path to vlc.exe in the command, then I get this error: "Command pattern is not setup correctly for this player."
YT2P 1.5 invokes VLC, but the link(s) sent to VLC do not appear to work any longer. If possible, I'll stay with Firefox 39.0 in the hopes of somhow keeping stutter-less Flash-based playback. And I'll never upgrade to anything newer beyond Firefox 43.0. Because cookie prompts.
Therefore, would it be possible to have support for older Firefox, at least going back to 38.8esr? I recently learned about some users who use even older ESR versions of Firefox, but that sounds like a stretch.

The so called 'fancy links' are always bloodred! Isn't it possible to make it a different color? It would be nice if the color could be chosen. In fact I would like no color at all.

I recently got an update to YT2P version 2.1 on Firefox 39.0, so the particular issue of availability to (slightly older) browsers is resolved. Thanks.

@Infobott Your issue is unrelated to the one I started, so you could start a new issue related to link colour.


AIY Speech V2 LED Example

It looks like with the new speech bonnet, the code to control the LEDs needs to be revised. I've gone through and tried to use the Vision LEDs example, however, there are a couple colors that are not behaving the same, noticeably 'red'.
How can I port the _led code for the speech hat to use with the new arcade button led?

That's been fixed in the latest code update, please take a look at board.py:
https://github.com/google/aiyprojects-raspbian/blob/5c74ce5154fe9ecdab5901e01b567d47fbdc068f/src/aiy/board.py#L286-L298


Does this project have a license?

Hello, I am interesting in packaging this module for nixos. But in order to do that, the project needs a license. Would it be possible to add a license? Thank you!

Oh I stopped supporting it, I change from Chargepoint to TechnoVE

Are you looking for a new maintainer? I could volunteer to take it over if so :) Thanks for adding the license in the meantime.


adding ubuntu 23.04 build support

Describe your PR, what does it fix/add?
This pull request goes together with the pull request in hyprland docs for adding proper and easy instructions for building in ubuntu 23.04
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
https://github.com/hyprwm/Hyprland/issues/2195
Is it ready for merging, or does it need work?
no, needs solving the previous bug for it to work well

You're hardcoding some random build for ubuntu, locking deps, for an OS we don't support for obvious reasons + are not willing to support.

You're hardcoding some random build for ubuntu, locking deps, for an OS we don't support for obvious reasons + are not willing to support.

basically are the project dependencies, and as is part of my dayly use, im willing to support, i got plenty of time for that.
then as is always a pain to look manually for the deps when they are not in a online guide.
what are your suggestions for improving this with the objective of merging

what do i need to explain/change

what are your suggestions for improving this with the objective of merging

There is no point, as none of the maintainers are willing to support ubuntu pipelines. Feel free to make your own repo for hyprland builds or w/e is the equivalent of arch AUR/fedora COPR/alike

i mean, its simple, you dont need to, its only a documentation issue, lets only merge the hyprland wiki with the change of the script is disponible in my repo only, and if in the future hyprland starts to use a dependency which brakes ubuntu if installed i can put in the wiki that you can only build lets say until x or y version on this ubuntu version
i think is important to do this cuz ubuntu is yet a popular distro and this new version adds quite nice changes to gnome and to the system in general also the nvidia support is great, which is going to attract all kinds of users.
since is posible and fully operational but quite tedious without a guide to use and build hyprland in ubuntu 23.04
would be great to have it, with the clear specification that hyprland does not intend to support ubuntu pipelines
also the message that this build guide can break in the future, in the case of that im willing to maintain the instructions and put any warning or advice if needed
i personally dont see a reaon why that cannot be done

If you want a note for ubuntu with a clear warning, sure, put it under the installation wiki part where there is a big combo with all the distros:

And not trash the main repo with platform-dependent scripts that we won't maintain.
Look at the fedora tab, and do the same for ubuntu. Make your gist, repo, whatever, and put a link to it there.
Ref: https://wiki.hyprland.org/Getting-Started/Installation/

Perfect!!, That sonds very resonable to me, and a great reason for no having ubuntu in the wiki, let me modify my pull request.
Thanks!!


The mat-form-field-invalid class does not reset properly with submit buttons

Bug, feature request, or proposal:
Bug
What is the expected behavior?
The mat-form-field-invalid class should be removed when calling myFormGroup.reset(), since this class normally only appears when the form control is invalid and touched.  This does occur if I use a button of type="button" and not type="submit".
What is the current behavior?
The mat-form-field-invalid class does not reset properly after calling myFormGroup.reset() after a submit button is triggered.
What are the steps to reproduce?
http://plnkr.co/edit/QGtqH8DDylViivcny1BA?p=preview
Steps to reproduce:
Scenario 1

Type anything into input field
Click the "Clear" button
Observe that the field is cleared and no form validation errors appear (it was reset properly)

Scenario 2

Now, type anything into the input field again
Click the "Submit" button (which is a submit button that calls formGroup.reset()
Observe that the field is cleared, but form validation triggers

Also note the value of mat-form-field-invalid in both scenarios.
What is the use-case or motivation for changing an existing behavior?
It is often common to reuse a form when entering multiple items into a system.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Tested using Angular 5.0.0, Material 5.0.0-rc.2, TypeScript 2.4.2
Is there anything else we should know?

I have access to the FormGroup.  How would I get access to the FormGroupDirective?

One of the ways can be seen in @willshowell's answer over here (I like this way): https://github.com/angular/material2/issues/4190#issuecomment-305222426
You can alson use:
<form #f="ngForm" (onSumbit)="doAnything(f)">

</form>

and in typescript:
doAnything(f: FormGroupDirective) {

}