text:
Enhancement: Logging Middleware

We've decided to include an optional logging middleware. The middleware should log incoming requests and out-going responses. It needs to be configurable:

The user should be able to dictate the values that are being logged and the logging ourput structure.
The middleware should support obfuscating data and excluding logging for specifc paths.
The middleware must use the logging config as a basis - i.e. register the logger as part of the config, and allow the user to decide on dictate what logger is actually used (e.g. picologger, stdlib, some structlogger etc.)

I am starting something.

Here some thinking about how to do it:

The middleware should log incoming requests and out-going responses

I will make a first PR with this capabilities (so very simple). I'll add two debug log, one on incoming request and the other on outgoing response. I am thinking using INFO level.
I will name the logger classicaly using name.

The user should be able to dictate the values that are being logged

Are you speaking of choosing if he logs both (incoming / outgoing) or only one of them or even none of them.
Another idea could be to choose what is logged:

url requested
content-type expected
browser used and other client information
response status
...

and the logging output structure.

I will add a format option that will override logging configuration for this logger.

The middleware should support obfuscating data and excluding logging for specifc paths.

I will add tow options:

first, a callable list to parse the str that will be logged (that's for obfuscating).
second a path exclusion list but I am unsure of the purpose of that. Shouldn't we log something minimal for that ?

The middleware must use the logging config as a basis - i.e. register the logger as part of the config, and allow the user to decide on dictate what logger is actually used (e.g. picologger, stdlib, some structlogger etc.)

I will add a logger_class in the configuration to load the correct one.
I am not an expert here, please feel free to complete your wish.

authors: Goldziher Swannbm

repo: starlite-api/starlite


text:
having issues running the autoupdate.bat

NOT found "termsrv.dll" version entry [10.0.19041.1949] in file "C:\Program Files\RDP Wrapper\rdpwrap.ini"!
keeps repeating waiting for network connection but it is connected to the network

We have had 5 computers now that can't update after the windows update so we have to restore them  back before the update

keeps repeating waiting for network connection but it is connected to the network

Not an RDP Wrapper issue. You can disable network check via editing the .bat file.
Discussion recommended in https://github.com/stascorp/rdpwrap/pull/1160

source: gharchive/issue

repo: stascorp/rdpwrap

license_type: permissive

license_source: bigquery


text:
Termserv not working 10.0.19041.1949

Need help....
All installed,look OK ,but no multiple connection.

Try restart service or reboot.
One termsrv.dll version = one opened issue!

Duplicate of https://github.com/stascorp/rdpwrap/issues/1968

Solved...
Tnx for advice.
After secund replacing in system32,work now.
Thanks.


text:
Windows 10 10.0.19044.3448 not supported

Does anyone have rdpwrap ini file?

Duplicate of https://github.com/stascorp/rdpwrap/issues/2289

authors: networkoper sashaqwert


text:
Not listening on Windows 10 Home 1803

Wrapper state 1.5.0.0 Installed
Service state 10.0.17134.1 Running
Listener State: Not Listening  [Fully Supported]
Was working before 1803 update

termsrv.zip

This is broken on Windows 10, 1803 and has been for near a month.
When will the app be updated?

Faced with issue on 1803 right after update (two hours ago)
system service won't start after applying rdpwrapper 1.6.2 with the newest ini file.
But I resolve it: just add Full Access to the both RDP Wrapper folder and rdpwrap.dll for user
Network Service
So now service is starting and working like a charm!

I added Network Service to the RDPWrapper  folder and the .dll inherited the security setting. However I still can't logon with the password defined in the RDP client. I have to leave the passwork blank and then supply it once the connection is made.

Was working before 1803 update

Then, do this:

After upgraded to 1803, open CMD with Admin rights and execute:
"copy C:\Windows.old\Windows\System32\rfx*.dll C:\Windows\System32"
Execute then (with Admin rights) the "uninstall.bat" of the RDP-Wrapper.
After that, re-install it ("install.bat").

With this I do the clean upgrade to 1803 HOME EDITION, and RDP returns to work!
Regards.

rfxvmt.zip
Windows 10 Home 1803 x64 working here:
1 Stop service:  remote desktop services
2 replace rfxvmt.dll in system32 with the attached dll
3 uninstall.bat
4 install.bat
Note: I am on 64 bit, but only replaced the rfxvmt.dll in system32 folder and did not touch anything in syswow64 folder

i got error access denied

When will the program be updated that just does it all for us? Can't it be done anymore?

Did everything people have suggested on here and still can't get RDPWrap to work on update 1803.

Same here, what is going on?

@jaxjexjox obviously update 1803 broke RDPWrap and I'm not sure who knows what exactly is causing it to break.  I do hope someone in the community finds out and issues a patch or new release.

According to this thread and the other one about this though, people have fixed it and got it working, I just can't get the right DLL going or whatever.     Surely if they can do it, the original package can be updated.

I've tried every single method the ones who have listed what they did to get RDPWrap working again.  None of them worked for me.  I stopped the remote desktop services, I gave network service full control to the RDPWrap folder and dll file, I've uninstalled and reinstalled, & none of it worked.

Are all you guys using Windows 10 HOME?

I just installed 1803 update and RDPWrap broke. I ran an admin CMD prompt and ran:
copy C:\Windows.old\Windows\System32\rfxvmt.dll C:\Windows\System32
Then did admin uninstall.bat and admin install.bat. RDP came back up immediately.

greensweater's fix worked for me, but I needed to run both installs before it worked properly.

@XythQ what do you mean both installs?

Get a look by : https://github.com/stascorp/rdpwrap/issues/501 (the end of the side/ the last bit-change is important) and before you have you unistall rdpwrapper
And then it works without rdpwrapper by multi rdp user
Sincerley

Not working for Windows 10 Home 64bit for me.
Neither greensweaters fix, nor patching termserv.dll according to #501 nor the combination of both.
I copied rfxvmt.dll, restarted, executed uninstall.bat, then install.bat. restarted. Still Service not running/not listening.
I patched termsrv.dll version 10.0.17134.1 as stated above, even tried a download of a patched version in case I messed patching up.
Still nothing :-/
Any hints for me?

64-bit Windows 10: https://github.com/stascorp/rdpwrap/files/1236856/rfxvmt.zip
32-bit Windows 10: https://github.com/stascorp/rdpwrap/files/1238499/rfxvmt.zip
These are from https://github.com/stascorp/rdpwrap/issues/194 ... I guess you could review that thread for additional help. There are many other links to rfxvmt.zip you could try (there is one in this thread). You may also need to run update.bat from the installer.

Will try that asap, thx dude!

I've got also the error with "Access denied", so i watched in the rdpconf.exe and disabled "single session per user" and now it works

So this still hasn't been updated?  I see it showing 2017 as latest version on the site?

Ran update.bat after replacing the dll as explained by @tnynyn and now my rdp wrap is working again on Win10 x64 Home.

Thanks @greensweater
I downloaded the 64 bit file. Stopped remote desktop service, copied the file to system32 restarted remote desktop service then ran update.bat then checked using rdpcheck.exe and now all working fine.
I had tried previously from other posts and could never get it working.

Closing as inactive (probably fixed in master branch).

authors: Furbach NMCowboy RDPWrapperUser Sp0oner Vasik76 Wootburger XythQ binarymaster greensweater jaxjexjox lars18th prototype2118 tnynyn westsmoker


text:
Fix issues with selecting fields in the content API

I don't love this approach, but it seems like it might be the only thing that works.
Switch to always getting all data from the database, then limiting whats returned by selectedQueryFields in the Entry class.
Fixes https://github.com/statamic/eloquent-driver/issues/26

@jasonvarga could you sanity check this when you have some time. I really don’t like getting all the data every query but I feel like there may be no other option.

can you see a reason why we shouldn't just get the full row each time

Performance, I guess. The same reason you don't just use select * all the time.
As for #96, we could add origin_id along with collection. Clearly collection isn't the only required field.

Sorry to be clear, initially I suggested "enforce the collection column is selected". What I really mean is that we need "enforce a list of columns".
If you do $query->get('title'), we should merge on collection, origin_id, maybe slug, etc.

Er wait I see what you're saying. The rest of the custom fields are all stored inside data. I forgot it worked that way! The important stuff is all dedicated columns. Ok I guess we just get them all 🤷

Glad it has you going round in circles too! I cant decide whats best either.

source: gharchive/pull-request

authors: jasonvarga ryanmitchell

repo: statamic/eloquent-driver

license: MIT

license_source: github-api


text:
"@state-adapt/angular": "^2.0.0", type errors

in change log there is no mention of breaking changes
ng: 16.2.9, typescript": "~5.1.3",
code:

@Component({
  selector: 'app-state-adapt-counter',
  standalone: true,
  imports: [CommonModule],
  providers: [defaultStoreProvider],
  template: `

count = {{count.state$ | async}}

double of count = {{count.doubleing$ | async}}

    <button (click)="count.increment(3)">Increment by 3</button>

    <button (click)="count.double()">Double</button>

    <button (click)="count.reset()">Reset</button>
  `,
  styles: [
  ]
})
export class StateAdaptCounterComponent {

  count = adapt(['count', 0], {
    increment: (state, payload: number) => state + payload,
    double: (state) => 2 * state,
    selectors: {
      doubleing: state => 2 * state
    },
  });

}

error:

X [ERROR] TS2322: Type '(state: (string | number)[]) => number' is not assignable to type '((state: (string | number)[], event: any, initialState: (string | number)[], selectorsCache: SelectorsCache) => (string | number)[]) | { doubleing: (state: (string | number)[]) => number; }'.
  Type '(state: (string | number)[]) => number' is not assignable to type '(state: (string | number)[], event: any, initialState: (string | number)[], selectorsCache: SelectorsCache) => (string | number)[]'.
    Type 'number' is not assignable to type '(string | number)[]'. [plugin angular-compiler]
...
X [ERROR] NG9: Property 'doubleing$' does not exist on type 'InitializedSmartStore<(string | number)[], Selectors<(string | number)[]>, ReactionsWithSelectors<(string | number)[], Selectors<(string | number)[]>>>'. [plugin angular-compiler]

The ! signifies a breaking change in Refactor(rxjs,react,angular,ngrx,ngxs)!: Simplify StateAdapt.adapt syntax. Yeah I should have made it obvious and linked to the issue itself in the change log. Here's the issue describing the changes: https://github.com/state-adapt/state-adapt/issues/45
I still haven't published the updated docs. I'll update the changelog when I do that.

The updated docs have been published https://state-adapt.github.io
With a link to the old docs on the sidebar for reference
And I've added more detail to the changelog and will do the same for future breaking changes (none planned at this point) https://github.com/state-adapt/state-adapt/blob/main/CHANGELOG.md

repo: state-adapt/state-adapt


text:
[BUG] > 1 time connections causing app crash

Issue description:
When the app run camera could be connected only once. When it's disconnected and connected again - the app will crash.
Environment:

Application version: 0.2.1
Device: Samsung Galaxy S8 SM-G950U1
OS: Android 7.0 Nougat
Camera: Intel Realsense D415 camera
Camera firmware: 5.11.1.100

Steps to reproduce:

Start the app
Connect the camera
Push ok in new device connected window
See that stream is started
Disconnect the camera
Put it back again
Push ok in new device connected window
App crashed

Expected behavior
There should be a way to connect/disconnect the camera multiple times without any issues.
Additional context
Related to the pipeline start/stop. During error it's not stopped and the next start will cause exceptions.

Fixed


text:
Refactor output buttons component

Move output-relevant buttons currently associated with input to a custom renderer and make available buttons as per execution state.
Not a strict mockup/design (mainly showcasing issue):

Can we identify which status bar items should be in the input vs. output cell? A premature list could be:
Input:

language (e.g. Shell Script)
Copy

Output:

Open Terminal (PID: XXX)
CLI
Background Task flag / Stop Task (merge both into one, include a "running" image signalling the user that this thing is running)
Copy

Did I miss something?

Sorry, overlooked this comment. The Input/Output partition looks correct to me 👍 .

I created a draft PR in #114, before moving on I would like to resolve the following questions:

I implemented this on top of the pseudo terminal experiment, as also implementing it for the "interactive" execution is not trivial and wouldn't provide lasting value
therefore I removed the CLI button which gets obsolete as everything is now run through the Runme CLI
the "Open Terminal" button currently only opens the terminal but won't close it, do we want that?

I implemented this on top of the pseudo terminal experiment, as also implementing it for the "interactive" execution is not trivial and wouldn't provide lasting value

This makes sense to me as long as we have a handle on what it would take to release the pseudo terminal. In other words, we want to avoid entanglement if it's not absolutely necessary. Not clear to me where we stand on this point. Let's go over this as part of planning tmrw.

therefore I removed the CLI button which gets obsolete as everything is now run through the Runme CLI

We need to be careful to not change the semantic of execution unilaterally. Now that the rearchitected exec engine is shaping up, let's see if we are ready do decide that the CLI will be the API client for both terminal & extension. I assume, there is not limitation on the pseudo terminal's part whether it'll spawn a CLI process or talk to the GRPC interface directly. Let's confirm. - In any case, removing the CLI button for the time being is fine.

the "Open Terminal" button currently only opens the terminal but won't close it, do we want that?

If it's possible to have a "Close Terminal" button when the terminal is open and vice versa. I think we do want that. However, the last time I checked (few months ago), it wasn't possible to query the open/close state of individual terminals.

In order to make updates to the output cell we need to keep the task running, therefore the "Stop Task" button seems useless given users already have the ability to press the stop button, remove it?

Yes, for non-background tasks that does make sense. However, we should keep it for background tasks, no?

as long as we have a handle on what it would take to release the pseudo terminal

In order to get the new pseudo terminal out of the door we need the execution engine to do all the work.

I assume, there is not limitation on the pseudo terminal's part whether it'll spawn a CLI process or talk to the GRPC interface directly. Let's confirm.

As far as I understood we want to have the CLI be the main process for executing cells in a runme notebook. I doubt there is much value implementing an intermediate solution for this and focus on getting the execution engine out of the door.

However, the last time I checked (few months ago), it wasn't possible to query the open/close state of individual terminals.

Seems like #125 solves this issue 🙌

However, we should keep it for background tasks, no?

Yes.

Added a blocked label until we decide how we want to move forward. IMO focusing on the execution engine and rethink our approach on these output buttons is the way to go. If there are any small improvements I can do for the meantime, let me know.

👍 makes sense to mark it as blocked for now. Need to dig in a little to figure out next steps which will likely not happen for some time.

I will un-assign me from this. It should be revisited once the execution engine is in place.

Closing this as superseded.

authors: christian-bromann sourishkrout

repo: stateful/vscode-runme


text:
Bug:  some "in" guard state references do not work

Description
in guards, similar to target, reference another state via a state expression. While this works when using the referenced state's own id and for states without an id via their "global path" (#rootNodeId.topLevelStateId.nestedStateId), it does not work for states with id via their global path, nor does it work to reference a state via a non-root parent with an id.
Maybe this is intended behavior in parallel states? I cannot really imagine, because the direct reference works across parallel nodes, so why should a non-root parent not be addressable?
Expected result
For nested states a, b and c with id idA, idB and idC, respectively, all these state reference expressions should work as the value of an in guard:

#idA.b.c
#idB.c
#idC

Actual result

only works when c has no id (then, of course, 3. is not possible)
never works
always works

Reproduction
https://stately.ai/viz/fafd7a49-e09e-4ed0-b420-3f8c5c0fc2db
Additional context
The linked XState Visualizer example shows four cases:

EVENT_GLOBAL - a working global path reference (no id on sub-state)
EVENT_DIRECT - a working direct reference
EVENT_GLOBAL_BUG - a buggy global path reference (id on sub-state)
EVENT_NESTED_BUG - a buggy path reference starting from a "middle" nested state with an id

In the simulation, you can see that only when hovering over 1. and 2., the follow-up state stateA2 is highlighted. When hovering over 3. or 4., nothing happens. When you click on 3. or 4., an error message is shown that the respective state reference could not be resolved.

If someone could give me code pointers and maybe answer some onboarding questions, I'd be willing to help fix this.

If you want this to be based on the SCMXL standard, here is what they say about in:

The set of operators in conditional expressions varies depending on the data model, but all data models must support the 'In()' predicate, which takes a state ID as its argument and returns true if the state machine is in that state. This predicate allows coordination among parallel regions.

https://www.w3.org/TR/scxml/#ConditionalExpressions
So in the spec, In is an operator to be used in any guard, not a dedicated property like in xstate JSON. Anyway, since it refers to a state (by ID), xstate JSON should use the same reference syntax as anywhere else.
Are you concerned that this would be a breaking change?

xstate@beta has been released, and the stateIn(...) guard is to be used instead of in:
on: {
  EVENT: {
-   in: '#someState'
+   guard: stateIn('#someState')
  }
}

authors: davidkpiano fwienber

repo: statelyai/xstate


text:
@xstate/test warns on final states and onDone handlers

Description

If your test machine uses the pattern where you have a type : "final" state and its parent uses an onDone the implicit event fired by the final state will be logged as a warning unless you create a definition for it in .withEvents().
Expected Result

No warning, it's not a normal event and doesn't need a user implementation.
Actual Result

Warning in the console about an even that clearly wasn't something the user intended to require an implementation of.
Reproduction
https://codesandbox.io/s/xstatetest-ondone-warning-g26k1
Additional context

@xstate/test@0.3.0
xstate@4.7.8

Repro Code
const { Machine: createMachine } = require("xstate");
const { createModel } = require("@xstate/test");

const machine = createMachine({
  id: "menu",
  initial: "initial",
  states: {
    initial: {
      initial: "inner1",

      states: {
        inner1: {
          on: {
            INNER2: "inner2"
          }
        },

        inner2: {
          on: {
            DONE: "done"
          }
        },

        done: {
          type: "final"
        }
      },

      onDone: "later"
    },

    later: {}
  }
});

const model = createModel(machine).withEvents({
  INNER2: {},
  DONE: {}

  // Uncommenting this will silence the warning
  // "done.state.menu.initial" : {}
});

const execute = async plans => {
  plans.forEach(plan => {
    plan.paths.forEach(async path => {
      console.log(`${plan.description}\n\t${path.description}`);
      await path.test({ ctx: "foo" });
    });
  });
};

(async function() {
  await execute(model.getSimplePathPlans());
})();

Warning generated by the repro
Missing config for event "done.state.menu.initial".

I added some logging of the plans and paths and noticed something that seems strange.
reaches state: "#menu.initial.inner1"
    via
reaches state: "#menu.later"
    via done.state.menu.initial
    via INNER2 → DONE
    via INNER2 → done.state.menu.initial
reaches state: "#menu.initial.inner2"
    via INNER2

Unless I'm misunderstanding the output here it thinks that #menu.initial → #menu.later and #menu.initial.inner2 → #menu.later are both possible via done.state.menu.initial. That seems wrong?

I'm seeing the same thing - I can't get generated test cases to pass because there is no way to trigger the "done.state.*" transitions directly; instead, it should be completing the child statechart.

I am running into the same issue. For my test paths generated with xstate/test, my tests that have paths with done.state.* fail and there is no way to trigger transitions for that state. Has there been any update on this?

This warning no longer occurs on @xstate/test@alpha: https://codesandbox.io/s/xstate-test-ondone-warning-forked-5krdu3?file=/src/index.js

authors: davidkpiano jamestharpe patel-jeel92 tivac


text:
Write a method to let it not log the information?

I'm seeing the same thing. When an error is thrown, its stack trace is being printed before the exception is thrown. I want to catch that exception when I know the error (for example, when the port is already in use) to keep the console clean. Fleck provides a LogAction function to customize the logging, but it doesn't take effect; it logs anyway. Is this the expected behavior?

repo: statianzo/Fleck


text:
sh: 90: local: 2.22.1: bad variable name

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
curl --proto '=https' --tlsv1.2 -sSfL https://get.static-web-server.net | sh

Logs/Screenshots
If applicable, add logs or screenshots to help explain your problem.

Environment and Specs

static-web-server: version from get.static-web-server.net
OS: Linux 4.19.0-16-amd64 x86_64

@kzhui125 can you please test the bugfix https://github.com/static-web-server/static-web-server/pull/274 in your Linux? Use the following command:
curl --proto '=https' --tlsv1.2 -sSfL https://raw.githubusercontent.com/static-web-server/static-web-server/fix-installer-script/scripts/installer.sh | sh

Resolved in v2.23.0

repo: static-web-server/static-web-server


text:
[RFC]: refactor blas/ext/base/dcusumpw to follow current project conventions

Description
This RFC proposes refactoring @stdlib/blas/ext/base/dcusumpw to follow current project conventions.
These conventions are outlined in more detail in https://github.com/stdlib-js/stdlib/issues/788. Namely, we want to migrate from C++ add-on interfaces to C add-on interfaces and then make various style and simplification changes.
Some example commits which perform most, if not all, of the requested changes include https://github.com/stdlib-js/stdlib/commit/d66d80cfcbab1da95cb5ae8cfae4a1fc4c5129d1 (saxpy),  https://github.com/stdlib-js/stdlib/commit/3c82fe7b6c0f9fd6ef02c86d6e2024d708ac8c14 (cswap), and https://github.com/stdlib-js/stdlib/commit/5b15c9675be434c0e11b1ec1e1f78d7b089abbcd (dasum).
Related Issues
Related issues #788, #1152.
Questions
No.
Other
No.
Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

I'm working on this.

authors: Planeshifter itskdhere

repo: stdlib-js/stdlib


text:
Qt Creator: Cannot open file 'QWKQuickd.lib'

I have recently tried to build qwindowkit to use with Qt Quick application. But I have noticed there no flies related to QWKQuick.pri
include(C:/dev/QWindowKit/share/QWindowKit/qmake/QWKQuick.pri)

As result, an error in Qt Creator:
:-1: error: LNK1104: cannot open file 'QWKQuickd.lib'
I've discovered there no files such 'QWKQuickd.lib' or 'QWKQuick.lib' in installation folder.
Maybe I made some mistakes on building?
There is my build prompt for reference:
c:\git\qwindowkit>cmake -B build -S . -DCMAKE_INSTALL_PREFIX=/path/install   -G "Ninja Multi-Config" -DQt6_DIR="C:/Qt/6.4.3/msvc2019_64/lib/cmake/Qt6"

c:\git\qwindowkit>cmake --build build --target install --config Debug

c:\git\qwindowkit>cmake --build build --target install --config Release

-DQWINDOWKIT_BUILD_QUICK=TRUE

-DQWINDOWKIT_BUILD_QUICK=TRUE

Thank you. Would try it and report you back

repo: stdware/qwindowkit


text:
Feature: Universal ABI for v8/NodeJS/Chromium

[ ] - Improve node-capn

[ ] sbffi into native capn build.

[ ] - Integrate CapnProto into v8

[ ] expose it in Node Chromium

This is cloesd in favor of the stealify component system which will also offer capn

repo: stealify/stealify


text:
视图中目前不支持子表字段的属性选择，默认指向了name属性

视图中目前不支持子表字段的属性选择，默认指向了name属性，比如联系人列表只能显示出单位名称，不能显示出单位电话，单位优先级，单位所有城市等属性。

saleforce也不支持，可以通过关联链接点击进入看到所有相关属性

repo: steedos/creator


text:
Settings are present and set up, but nothing shows in "Source Control" and all command palette options say "not found"

For example, "Perforce: Show Output" produces this error:
command 'perforce.showOutput' not found
I can't see any evidence of the extension working, except for the settings section and the entries in the command palette. Either something is wrong, or this extension is really confusing to set up.
VS code info:
Version: 1.30.2 (system setup)
Commit: 61122f88f0bf01e2ac16bdb9e1bc4571755f5bd8
Date: 2019-01-07T22:54:13.295Z
Electron: 2.0.12
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
OS: Windows_NT x64 10.0.17134
No other extensions installed.
Please help :)

@stef-levesque: Please help! I am facing the exact same issue, it's like not having the plug-in at all :(

Also having this issue. Did a VS Code update break it maybe?

Are you sure you have a folder opened in VSCode when trying to execute the extension ?
Can you try to change the activation mode to Always in your settings?

Yes.
Setting to "always" fixed it. Not sure what happened…

I also have this issue. Setting the activation mode to "Always" does not work.

Not sure why @stef-levesque closed the issue, it's clearly not resolved…

authors: Deco NSExceptional VinayGupta23 cyliang stef-levesque

repo: stef-levesque/vscode-perforce


text:
fix friendbot sequence number issues

I've tested this locally and it refreshes the friendbot sequence number correctly now. One of the issues we had was that it was not correctly checking for the tx_bad_seq, which is fixed.
On another note, I’m seeing a lot of 504 gateway timeouts (error decoding horizon.Problem: EOF) and "400 Bad Request" as responses from horizon when I’m testing locally. I don't think this can be fixed since we naturally want to push multiple txns per ledger with friendbot and cannot preempt these responses (at least not in a way that I'm currently aware). We are seeing this now as opposed to earlier because this is the first time friendbot is hitting the horizon server directly as opposed to going through the txSub sub-system.
@nullstyle

I'm going to go ahead and merge this.  While these changes don't solve the root issues around friendbot (it still breaks pretty trivially under lots of concurrent requests) this PR should improve things somewhat while I do a round of improvements in transaction submission and testing.

authors: nikhilsaraf nullstyle

repo: stellar/go


text:
UTF-8 support

Updates stellar-base to 0.5.6 which contains UTF-8 support in memo_text.

+1

LGTM

authors: bartekn irisli nullstyle

repo: stellar/laboratory


text:
Upgrading smartcontract

Bounty Submission for #626
Hello @briwylde08 This is ready for review

Hi @ElliotFriend @briwylde08 , I have resolved the merge conflict and updated file folder structure.

Thanks for fixing the conflicts! I'm guessing we'll get this reviewed next week. It's a holiday tomorrow in the US so most of the team has a short week. 🙏

@briwylde08 alright, looking forward to it.

Also, I don't know why Github is telling us that 243 files have been changed 🤣 I can see by comparing here that it's really only the one file. So... 🤷🏻♂

Hi @ElliotFriend , I will definitely make up this changes and I push up soon.
For the 243 files changed 😂, I have worked on this tasks before the changes you guys made in the codebase, so I had to pull all those changes in the branches I'm working on the different tasks assigned to me.

HI @ElliotFriend , I made changes to the file you reviewed and I have pushed the updated format.

@ElliotFriend I will make changes to them now and push up in few minutes time.

Hi @ElliotFriend , I made a push now with the updated changes.

@KodeSage Looks good! Once the build workflow finishes successfully, I'll give an approval.

Thank you so much for your work here @KodeSage! I'll merge this and we can take the award conversation back to the corresponding issue.

authors: ElliotFriend KodeSage briwylde08

repo: stellar/stellar-docs


text:
Add support for removing columns

Similar to try db.run(users.addColumn(suffix)), it would be useful to also be able to delete columns using syntax like try db.run(users.deleteColumn(suffix))

In the absence of such a function, something like the following can be ran:
try db.execute("""
    BEGIN TRANSACTION;
    ALTER TABLE your-table
    DROP COLUMN your-column;
    COMMIT TRANSACTION;
    """
)

Thank you! 🙏

license: mit


text:
Save Base64encoded string

Hi,
How can I store my image that I cast it to Base64encode in database?bigger than string I need.
Thanks

not a bug, closing.


text:
Is it possible to change the behavior of Android system back softkey

Currently, by pressing the Android system back softkey, it will quit the entire wizard activity.
However, is there possibility, that we can change such behavior in the way.

If we are in step 2, step 3, ... , pressing Android system back softkey, will enable us to back to page step 1, step 2, ...

If we are in step 1, pressing Android system back softkey, will enable us to quit the entire wizard activity.

Thank you

Hi,
The StepperLayout does not handle presses on the Back button itself. This is actually handled by the Activity/Fragment it is in. In the sample app it is handled by overriding onBackPressed() method: https://github.com/stepstone-tech/android-material-stepper/blob/master/sample/src/main/java/com/stepstone/stepper/sample/AbstractStepperActivity.java and simply closing the Activity if we are on step 1.
However, this behavior can be changed as you please :) I hope this answers your question.

Opps. Thanks for the great tip. I think that solves my concern. Thanks.

Hi Please give me solution
@yccheok please give me code if your code is work

@zawadz88
https://github.com/stepstone-tech/android-material-stepper/blob/master/sample/src/main/java/com/stepstone/stepper/sample/AbstractStepperActivity.java
this link is not working

Hi,
This class was converted to Kotlin. New link: https://github.com/stepstone-tech/android-material-stepper/blob/master/sample/src/main/java/com/stepstone/stepper/sample/AbstractStepperActivity.kt

authors: chiragmobibox yccheok zawadz88

repo: stepstone-tech/android-material-stepper


text:
Updated to support UE5.2

Changed use of uint8 to bool, fixing compiler warnings for conversion.
Added constructors to structs where there weren't any.
Updated Constructor initialisation lists to be complete to matching properties and reordered where properties were warned against being initialized after others.
Added use of Texture/Material Getters and Setters.
Fixed getting of D3D Native Resource (in StereolabsTexture.cpp). This accompanies the change in the D3D RHI in UE5.2 update.

Thanks a lot for the PR ! :)

authors: Bvallon-sl gaslightgames


text:
🛑 Stethoscope Docs is down

In 9b85b84, Stethoscope Docs (https://stethoscope.js.org) was down:

HTTP code: 530
Response time: 46 ms

Resolved: Stethoscope Docs is back up in a3b29a5.

repo: stethoscope-js/status


text:
[FR] add luasocket into AndroLua

this is a very widely used module, must have.

done in my fork: https://github.com/ildar/AndroLua/commit/e0e021725ec5f9d4ee595ca8216a7b5c3dbc39ae

repo: stevedonovan/AndroLua


text:
Create 0000-01-03-stevek180.md

Adding slide three

Well, i dunno....  too many slides?


text:
Feat/vectorized calculations

uses numpy vectorization to perform some calculations.
A little help on the way to #5

not sure I understand why the python 3.7 tests are failing on travis-ci.com but succeeding travis-ci.org (also on all my local tests)

Here's a before and after comparison of some OVRO LWA calculation timings.
Largest increase comes from the 2D calculation, but I have noticed some good speed ups in the phasing UVW one too in some HERA calculations.
Before:

After:

Oh, you should add yourself to the authors list and put the changes in the changelog...

authors: mkolopanis steven-murray


text:
Getting tagged Photos of You

Hi!  I was able to display images from my Instagram with a specific hashtag.  However, I'm wondering if it is possible to get images where in my account is tagged (images from Photos of You)?

It's not possible with basic permissions and in sandbox mode, and that's all that will be issued going forward. Sadly, can't be done!

authors: benjamin-hull jmlawas


text:
Video from feed not showing

I implemented instafeed to a website I am working on and I found an issue. There are multiple videos on the feed but one of the videos is missing and I don't know why.
This is the instagram url of the company: https://www.instagram.com/urbanelectriccompany/
This is where I implemented instafeed: https://www.urban-electric.com/archive
This is the code I am using for instafeed:
var instagramFeed = new Instafeed({
	userId: *****,
	accessToken: *****,
	get: 'user',
	target: 'instagram-feed-inner',
	resolution: 'standard_resolution',
	sortBy: 'most-recent',
	template: '{{model.created_time}}Click to open on Instagram{{model.template}}',
	filter: function(image) {
		if (image.type == 'image') {
			image.template = '';
		} else {
			image.template = '<video autoplay muted loop><source src="' + image.videos.standard_resolution.url + '" type="video/mp4"/></video>';
		}
		return true;
	}
});
instagramFeed.run();

I am using the latest version of Instafeed.

Hi @sorinu26 ,
I'm currently getting an access_token is invalid error on your example page.
Assuming you can fix that, what you're describing sounds like a symptom of running in Sandbox mode at Instagram: Sandbox mode (which all apps created in the past ~2 years are running in) limits you to the 20 most recent posts, anything outside of that will not be returned via the API.
Also, just wanted to make sure you're aware of the coming closure of the legacy Instagram API that instafeed.js uses: This is happening in 'Early 2020' (this is the schedule that IG has stated), so your integration will stop working then.


text:
Extension causes high cpu load/ gets stuck loading the preview window

<auto stuff>

Issue Type: Performance
Extension Name: shader-toy
Extension Version: 0.10.14
OS Version: Windows_NT x64 10.0.19042
VSCode version: 1.51.1

stevensona.shader-toy-unresponsive.cpuprofile.txt
Find more details here: https://github.com/microsoft/vscode/wiki/Explain-extension-causes-high-cpu-load
</auto stuff>
So in short I was trying to open the preview window of one of my projects and the plugin refuses to start.
The other log says something like:
[2020-12-09 17:40:09.820] [renderer1] [error] Illegal argument: Error: Illegal argument
    at Object.t.illegalArgument (file:///C:/Users/leon2/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:50:46)
    at file:///C:/Users/leon2/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2045:110
    at file:///C:/Users/leon2/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1982:363
    at d.invokeFunction (file:///C:/Users/leon2/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1724:588)
    at u._tryExecuteCommand (file:///C:/Users/leon2/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:4151:37)
    at file:///C:/Users/leon2/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:4150:774
[2020-12-09 17:40:24.350] [renderer1] [warning] UNRESPONSIVE extension host, 'stevensona.shader-toy' took 100% of 6428ms, saved PROFILE here: 'c:\Users\leon2\AppData\Local\Temp\exthost-b6b883.cpuprofile' [{"id":"gc","total":3000,"percentage":0},{"id":"program","total":8000,"percentage":0},{"id":"stevensona.shader-toy","total":6417000,"percentage":100}]
[2020-12-09 17:40:32.658] [renderer1] [warning] {}
[2020-12-09 17:41:12.936] [renderer1] [error] Failed opening file c:\Users\leon2\AppData\Roaming\Code\logs\20201209T173951\exthost1\exthost.log for writing: Permission denied: Error: Failed opening file c:\Users\leon2\AppData\Roaming\Code\logs\20201209T173951\exthost1\exthost.log for writing: Permission denied
    at createRotatingLogger (c:\Users\leon2\AppData\Local\Programs\Microsoft VS Code\resources\app\node_modules.asar\spdlog\index.js:34:9)
    at c:\Users\leon2\AppData\Local\Programs\Microsoft VS Code\resources\app\node_modules.asar\spdlog\index.js:27:7
    at c:\Users\leon2\AppData\Local\Programs\Microsoft VS Code\resources\app\node_modules.asar\mkdirp\index.js:48:26
    at FSReqCallback.oncomplete (fs.js:159:5)
[2020-12-09 17:41:41.556] [renderer1] [warning] UNRESPONSIVE extension host, 'stevensona.shader-toy' took 100% of 6625ms, saved PROFILE here: 'c:\Users\leon2\AppData\Local\Temp\exthost-33e175.cpuprofile' [{"id":"gc","total":7000,"percentage":0},{"id":"stevensona.shader-toy","total":6618000,"percentage":100}]

My File causing the Issue (sorta' long):
#iUniform float MinDist = 0.0005 in { 0.0001, 0.01 }
#iUniform float MaxDist = 15.0
#iUniform float reflectionThreshhold = 0.001 in {1e-5, 0.1}
#iUniform float zoom = 1.5 in {0, 10}
#iUniform float blend = 1 in {1, 10}
#iUniform float gamma = 1 in {0.1, 10}
#iChannel0 "self"

#define TWO_PI 6.28318530718
const vec3 sun = normalize(vec3(5,-10,10));

mat3 setCamera( in vec3 ro, in vec3 ta, float cr )
{
	vec3 cw = normalize(ta-ro);
	vec3 cp = vec3(sin(cr), cos(cr),0.0);
	vec3 cu = normalize( cross(cw,cp) );
	vec3 cv = normalize( cross(cu,cw) );
    return mat3( cu, cv, cw );
}

struct Material{
	float roughness;
	float reflectance;
	float diffuse;
	vec3 color;
};

struct Map{
	float dist;
	Material material;
};

struct Ray{
	vec3 p;
	float dist;
	int steps;
	Material material;
	vec3 normal;
};

const Material NONE = Material(0.,0.,0.,vec3(0));
const Material Floor = Material(
	.05,
	.3,
	.5,
	vec3(.2,.9,.2)
);
const Material Sphere = Material(
	0.1,
	0.7,
	0.2,
	vec3(0.9)
);

vec2 hash23(vec3 p3)
{
	p3 = fract(p3 * vec3(443.897, 441.423, 437.195));
    p3 += dot(p3, p3.yzx+19.19);
    return fract((p3.xx+p3.yz)*p3.zy);
}
vec2 hash2( float n ) { return fract(sin(vec2(n,n+1.0))*vec2(43758.5453123,22578.1459123)); }

Map map(vec3 p){
	float dFloor = p.y;
	float dSphere = length(p-vec3(0,1,0))-1.0;

	if (dFloor<dSphere){
		return Map(dFloor,Floor);
	}else{
		return Map(dSphere,Sphere);
	}
}

vec3 calcNormal(vec3 p)
{
    const float h = 0.0001; // replace by an appropriate value
    const vec2 k = vec2(1,-1);
    return normalize( k.xyy*map( p + k.xyy*h ).dist +
                      k.yyx*map( p + k.yyx*h ).dist +
                      k.yxy*map( p + k.yxy*h ).dist +
                      k.xxx*map( p + k.xxx*h ).dist );
}

Ray castRay(vec3 ro, vec3 rd){
	float dist = 0.;
	vec3 p = ro;
	int steps=0;
	for(;dist<MaxDist;steps++){
		Map map = map(p);
		if (map.dist<MinDist){
			return Ray(p,dist,steps,map.material,calcNormal(p));
		}
		dist += map.dist;
		p += rd * map.dist;
	}
	return Ray(p,dist,steps,NONE,vec3(0));
}
vec3 skyColor(vec3 rd){
	vec3 col = vec3(0.30, 0.36, 0.60) - (rd.y * 0.7);
	// sun glare
    // float glare = clamp( dot(sun,rd), 0.0, 1.0 );
    // col += 0.25*vec3(1.0,0.4,0.2)*pow( glare, 4.0 );
	return col;
}

vec3 diffuseColor(Ray ray){
	vec3 lv = -sun;
    vec3 ro = ray.p+ ray.normal*2.f*MinDist;
    Ray r = castRay(ro,lv);
    float b = (dot(ray.normal,lv));

	if(r.dist<MaxDist) b*= 0.2;
	b = max(b,0.);
	vec3 sunC = vec3(0.9, 0.9, 0.8) *b;
	vec3 ambient = vec3(0.03, 0.04, 0.1);

    vec3 col = ray.material.color * (sunC+ambient);

	return col;
}

vec3 applyRoughness(Ray r){
	float alpha = r.material.roughness;

	vec2 rand = hash23(r.p);
	float epsilon1 = rand.x;
	float epsilon2 = rand.y;

	float thetam = atan(alpha*sqrt(epsilon1),sqrt(1.-epsilon1));
	float phim = TWO_PI * epsilon2 * alpha;

	float phin = atan(r.normal.y,r.normal.x);
	float thetan = acos(r.normal.z);

	float theta = thetan + thetam;
	float phi = phin + phim;

	vec3 m = vec3(
		sin(theta) * cos(phi),
		sin(theta) * sin(phi),
		cos(theta)
	);

	return normalize(m);
}

vec3 getRefl(in Ray r,vec3 rd){
    float factor = 1.f;
    vec3 colorFactor = normalize(vec3(1));
    vec3 col = vec3(0);
    while (length(colorFactor)>reflectionThreshhold){
		if(r.material == NONE){
			col += colorFactor * skyColor(rd);
			break;
		}
		vec3 normal = applyRoughness(r);
		// r.normal = normal;
        vec3 colDiff = diffuseColor(r);
        col += colDiff * colorFactor;

        colorFactor *= r.material.reflectance * r.material.color;

        float s = dot(normal,rd);
        // the new reflected rd
        rd = normalize(rd - 2. * s * normal);

        r = castRay(r.p+r.normal*MinDist*2.f,rd);
    }
    return col;
}

vec3 color(vec3 ro,vec3 rd, out Ray ray){
	ray = castRay(ro,rd);
	//Sky
	if (ray.material==NONE){
		return skyColor(rd);
	}
	// Material Based
	return getRefl(ray,rd);
}

void colorCorection(inout vec3 col){
	// gamma correction
	col = pow(col,vec3(gamma));
}

void mainImage( out vec4 fragColor, in vec2 fragCoord ){
	vec2 o = hash2( float(iFrame) ) - 0.5;

    vec2 uv = (-iResolution.xy + 2.0*(fragCoord+o))/ iResolution.y;

    float th1 = (iMouse.x-.5*iResolution.x)/iResolution.x*8.f;
    float th2 = ((-iResolution.y+2.*iMouse.y)/iResolution.y+0.5)*2.;

    // uv.y+= asin(th2);

    vec3 ro = vec3(0,2,-4);

	ro *=  mat3(
        vec3( cos(th1), 0 , sin(th1)),
        vec3(       0, 1 , 0      ),
        vec3(-sin(th1), 0 , cos(th1))
    );

    vec3 ta = vec3(0,th2,0);
	mat3 ca = setCamera(ro,ta,0.);

	vec3 rd = ca * normalize( vec3(uv.xy,zoom));

	Ray r;
    vec3 col = clamp(color(ro,rd,r),0.f,1.f);

	colorCorection(col);

	mat4 oldCam = mat4( textureLod(iChannel0,vec2(0.5,0.5)/iResolution.xy, 0.0),
                        textureLod(iChannel0,vec2(1.5,0.5)/iResolution.xy, 0.0),
                        textureLod(iChannel0,vec2(2.5,0.5)/iResolution.xy, 0.0),
                        0.0, 0.0, 0.0, 1.0 );

    // world space
    vec4 wpos = vec4(ro + rd*r.dist,1.0);
    // camera space
    vec3 cpos = (wpos*oldCam).xyz; // note inverse multiply
    // ndc space
    vec2 npos = zoom * cpos.xy / cpos.z;
    // screen space
    vec2 spos = 0.5 + 0.5*npos*vec2(iResolution.y/iResolution.x,1.0);
    // undo dither
    spos -= o/iResolution.xy;
	// raster space
    vec2 rpos = spos * iResolution.xy;

    if( rpos.y<1.0 && rpos.x<3.0 )
    {
    }
	else
    {
        vec3 ocol = textureLod( iChannel0, spos, 0.0 ).xyz;
    	if( iFrame==0 ) ocol = col;
        col = mix( ocol, col, clamp(exp(-blend),0.,1.) );
    }

    if( fragCoord.y<1.0 && fragCoord.x<3.0 )
    {
        if( abs(fragCoord.x-2.5)<0.5 ) fragColor = vec4( ca[2], -dot(ca[2],ro) );
        if( abs(fragCoord.x-1.5)<0.5 ) fragColor = vec4( ca[1], -dot(ca[1],ro) );
        if( abs(fragCoord.x-0.5)<0.5 ) fragColor = vec4( ca[0], -dot(ca[0],ro) );
    }
    else
    {
        fragColor = vec4( col, 1.0 );
    }
}

Other shaders seem to work, so propably my shader, but no error output is given

Thanks for reporting this, I'll take a look at it on the weekend. Probably getting stuck in the parser because of some corner case.

repo: stevensona/shader-toy


text:
Create new input fields in line with style guidelines

Input fields should have a thin line around them, same bg as page and the ends should be rounded with half circles. Much like the design of the secondary button.

The input fields in the agreement distribution, sum and date picker have the right style. but are not general components. They should be extracted as general components so that we may use them other places.

repo: stiftelsen-effekt/main-site


text:
Security: Bump minimist to 1.2.2

This adresses a moderate severity prototype pollution attack in minimist <1.2.2 https://app.snyk.io/vuln/SNYK-JS-MINIMIST-559764

The appveyor CI check fails with Error: incorrect header check while installing elm. I'm unsure why this is happening, but it's happened on several other PRs so it seems to be an issues with the build pipeline. @stil4m maybe you've seen this error message before?

https://ci.appveyor.com/project/stil4m/elm-analyse/builds/31489839


text:
Google Analytics events cap

It looks like Google Analytics caps events at 500 per user session (and for physical installations it often seems to think there is only 1 session for the whole day). Wondering if there's a way to force a new session once in a while, so all events are captured?
Maybe with something like sessionControl here?
https://github.com/peaksandpies/universal-analytics/blob/master/AcceptableParams.md

Interesting. Something like this in logging.js seems like it might work.
// Log to Google Analytics.

// Restart a session every 500 events, see issue #40
this._google.eventCount = this._google.eventCount || 0;
var params = {};
if(this._google.eventCount >= 500) {
    params.sessionControl = 'start';
}

this._google.event(data.Category, data.Action, data.Label, data.Value, params);
var queue = _.clone(this._google._queue);
this._google.send(_.bind(function(error) {
    if (!error) {
        this._google.eventCount++;
        return;
    }

    if (error.code === 'ENOTFOUND') {
        // Couldn't connect -- replace the queue and try next time.
        // https://github.com/peaksandpies/universal-analytics/issues/12
        this._google._queue = queue;
    } else {
        // Something else bad happened.
        logger.warn('Error with Google Analytics', error);
    }
}, this));

The version of universal-analytics might need to be updated too... it's pretty old now.
Wanna try it out?

Heh, looks like I complained about this at some point myself.
https://github.com/peaksandpies/universal-analytics/issues/12#issuecomment-58433108

Haha... I was wondering why we didn't notice it before. Okay I'll try out sessionControl and update if that works. Thanks!

It seems to be working (I'm seeing more than 1 session over time)! I also updated universal-analytics version. Going to test it out a couple of days, then will submit a PR if all looks good.

Fixed in 1.7.8.

authors: LRitesh endquote

repo: stimulant/ampm


text:
opanAL on MacOS

I'm running into an issue which appears to be installing openAL on MacOS 10.14.6 (Mojave). When I go through the stitchEm install scripts it's failing on openAL.

sudo port install opencv glew gsed jpeg libpng openal           tiff faac faad2 ceres-solver glfw ffmpeg glm OpenEXR

--->  Attempting to fetch openal-1.3_0.darwin_18.x86_64.tbz2 from https://packages.macports.org/openal
--->  Attempting to fetch openal-1.3_0.darwin_18.x86_64.tbz2 from http://aus.us.packages.macports.org/macports/packages/openal
--->  Attempting to fetch openal-1.3_0.darwin_18.x86_64.tbz2 from http://ywg.ca.packages.macports.org/mirror/macports/packages/openal
--->  Building openal
Error: Failed to build openal: command execution failed
Error: See /opt/local/var/macports/logs/_opt_local_var_macports_sources_rsync.macports.org_macports_release_tarballs_ports_audio_openal/openal/main.log for details.
Error: Follow https://guide.macports.org/#project.tickets to report a bug.
Error: Processing of port openal failed
Which of course causes issues when I try to cmake.
Looking at main.log as indicated in the terminal error the bit that seems the most relevant is the 404 error:
:debug:archivefetch Fetching archive failed: The requested URL returned error: 404 Not Found
:msg:archivefetch --->  Attempting to fetch openal-1.3_0.darwin_18.x86_64.tbz2 from http://aus.us.packages.macports.org/macports/packages/openal
I'm not familiar enough with where I should be going to get openAL when it fails with macports. I've seen some mention that it's already installed on macOS and I can't find the developer section at creative that appeared to have downloads and openAL.org the SDK download is an .exe
I think Steve in issue #27 ran into the same exact thing I've been facing but I'm not sure how he got openAL installed to finish the cmake.
Any suggestions?
Mike

Hi Mike,
just remove openal from the install invocation. The package is no longer needed to build the software. It seems there was an oversight in the build documentation when OpenAL support was removed.

authors: aspentreemedia w-m

repo: stitchEm/stitchEm


text:
Fix type error.

It does't work on php 7.2.
PHP Fatal error:  Uncaught TypeError: Argument 2 passed to Stk2k\\NetDriver\\Util\\CharsetUtil::convertEncoding() must be of the type string, null given, called in /var/www/html/crypto-currency/vendor/stk2k/net-driver/src/Http/HttpResponse.php on line 43

Pull Request Test Coverage Report for Build 32

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

Totals

Change from base Build 29:
0.0%

Covered Lines:
232

Relevant Lines:
430

💛  - Coveralls

authors: coveralls sakurashiki


text:
USB-SERIAL_NOTES: add note to inform user to build in release mode

Fixes #286
Changes Proposed

[x] Add note on why one needs to build the example using the release profile

Great, thanks for the PR!


text:
refactor!: switch Node to enum-style

Goals

Avoid that consumers have to manually check the documentation in order to know when it is safe to unwrap node names/values
Make the API feel more rusty

Notes

Decided against the NodeElementAttribute / NodeElementChild enums as it would complicate the API significantly
The price for less complicated node name/value unwrapping – and hence more type safety – is a slightly harder to use API, but well worth it

Remaining work

[x] Add TryInto<T> implementations on node types to get a specific syn expression for node values, such as TryInto<Expr::Lit> for NodeText
[x] Fix tests

Feedback welcome.
Resolves https://github.com/stoically/syn-rsx/issues/16, Resolves https://github.com/stoically/syn-rsx/issues/15

Let me know if I can help. I agree with everything you said. And about the duplication part, I'm not that familiar with syn-rsx's internals, but we can implement the common stuff on the enum directly, no? And maybe have the different structs implement a common trait.
https://github.com/stoically/syn-rsx/issues/16#issuecomment-1279836683

I see the remaining work checklist. I might give some of those a crack, in the coming week.

Codecov Report

Merging #23 (650f2ad) into main (0f13493) will decrease coverage by 10.86%.
The diff coverage is 50.51%.

@@             Coverage Diff             @@
##             main      #23       +/-   ##
===========================================
- Coverage   84.74%   73.87%   -10.87%
===========================================
  Files           4        4
  Lines         367      356       -11
===========================================
- Hits          311      263       -48
- Misses         56       93       +37

Impacted Files
Coverage Δ

src/config.rs
62.50% <ø> (-37.50%)
:arrow_down:

src/lib.rs
50.00% <ø> (-19.24%)
:arrow_down:

src/node.rs
38.05% <34.24%> (-9.17%)
:arrow_down:

src/parser.rs
93.72% <100.00%> (-1.06%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I see the remaining work checklist. I might give some of those a crack, in the coming week.

Thanks, but it's already done. I'd be happy about a review tho. The node enum makes things slightly harder to access, but overall it feels more stable and rusty now I'd say.

Imma migrate part of my project to the node-enum branch and report back.

Force-push implements AsRef<Expr> for NodeValue so it gets easier to work with node value: less required usage of &*.

Force-push implements AsRef<Expr> for NodeValue so it gets easier to work with node value: less required usage of &*.

This is very welcome. I'm already using it.

Force-push renames NodeValue::value to NodeValue::expr, since a) that's what it actually is and b) if you have to access the expression directly, it's now node.value.expr instead of node.value.value.
Also adds From<Expr|ExprLit|ExprBlock> for NodeValue, primarily for internal parser usage.

makes sense. I was gonna suggest this. This may be futile by me, but value.value really annoyed me visually. And isn't implementing Deref (and DerefMut) and having the dingle field public for newtypes considered bad?
Could the expr be made private and the Deref, DerefMut implementations removed? And then implement From<&NodeValue> and From<NodeValue> for syn::Expr?
Maybe even pass Expr's methods through but I feel like that's not really needed since you should never really need to mutate the expr in place. But I could be wrong.

Right, NodeValue is a smart pointer and those generally don't expose the actual type directly. I wonder why that's anti-pattern tho. :thinking: In any case, force-push makes expr private.
I wonder tho, why would you prefer the From<T> implementations over the Deref one? The latter seems to be an often used pattern for smart pointers?

Force-push also adds From<NodeValue> for Expr and From<&'a NodeValue> for &'a Expr, because having more ways to get to the value.expr can't hurt either way, right?

Force-push also adds From<NodeValue> for Expr and From<&'a NodeValue> for &'a Expr, because having more ways to get to the value.expr can't hurt either way, right?

Yeah

I wonder tho, why would you prefer the From<T> implementations over the Deref one? The latter seems to be an often used pattern for smart pointers?

I guess I am not considering NodeValue a smart pointer. I'm considering it a newtype: it only really wraps Expr for semantics and for the implementation of traits.
And for newtypes implementing Deref is an anti-pattern.
Could you create a discussion for general architecture?
This isn't really related to the Node enum anymore.
I'm developing a web framework, heavily influenced by Qwik (resumability) and Lit (struct based, like Lit is class based). It's just a toy project, it's not even public yet. But it is complex enough that I feel I could have further suggestions.

Force-push renames NodeValue to NodeValueExpr so it's easier to recognize its intended purpose.
@Zizico2

Could you create a discussion for general architecture?

Did so here: https://github.com/stoically/syn-rsx/issues/26

I guess I am not considering NodeValue a smart pointer. I'm considering it a newtype: it only really wraps Expr for semantics and for the implementation of traits.

I feel like it's a tricky topic, but picked it up in the linked issue above. Happy to discuss that further.

I'm developing a web framework, heavily influenced by Qwik (resumability) and Lit (struct based, like Lit is class based). It's just a toy project, it's not even public yet. But it is complex enough that I feel I could have further suggestions.

Sounds interesting! I'd be interested to take a look once you publish something.

Released as v0.9.0-alpha.1

authors: Zizico2 codecov-commenter stoically

repo: stoically/syn-rsx


text:
isValid() function no longer provided for v7 of ip-address

Creation throws error instead
Signed-off-by: Kevin Cormier kcormier@redhat.com
https://github.com/stolostron/backlog/issues/24517

/cc @jeswanke

repo: stolostron/console


text:
ACM-14332 > Clusters page CSV export fixes extra space

ACM-14332 > Clusters page CSV export fixes extra space

/retest-required

/retest-required

/retest

/retest

/retest

/lgtm

/unhold


text:
[release-2.8]Ensure policies remain when hosting cluster is a hub

The bug was by deleting the hcp cluster, we see that all the ACM policies on the ACM hub are also deleted.
Signed-off-by: Yi Rae Kim yikim@redhat.com
(cherry picked from commit c89a259ee16d48951964fe4e26242f0cc543188c)

/unhold

/cherry-pick release-2.7

authors: mprahl yiraeChristineKim


text:
search addon migration

story: https://github.com/stolostron/backlog/issues/18488
Signed-off-by: Zhiwei Yin zyin@redhat.com

/retest

/assign @jlpadilla

/assign @xrajesh

/hold
waiting for https://github.com/stolostron/search-chart/pull/132  and https://github.com/stolostron/search-operator/pull/99 are merged.

/unhold

repo: stolostron/klusterlet-addon-controller


text:
remove the deprecated io/ioutil package

Signed-off-by: myan myan@redhat.com
Resolve the following issue when building the manager image:
SA1019: "io/ioutil" has been deprecated since Go 1.19: As of Go 1.16, the same functionality is now provided by package io or package os, and those implementations should be preferred in new code. See the specific function documentation for details. (staticcheck)
        "io/ioutil"
        ^

/cc @clyang82 @morvencao

Good catch. But it seems you need to ensure the code coverage is greater than 70% so that it can be merged.

repo: stolostron/multicluster-global-hub


text:
Update submariner-operator to v0.14.0-m1

to accommodate changes after operator-sdk upgrade.

api/submariner/v1alpha1 -> api/v1alpha1
pkg/client usage removed
other submariner projects are also updated.

Fixes: https://github.com/stolostron/submariner-addon/issues/481
Part of epic: https://github.com/submariner-io/enhancements/issues/81
Signed-off-by: Janki Chhatbar jchhatba@redhat.com

/assign @Jaanki

/cc @skitt @tpantelis

Closing in favor of https://github.com/stolostron/submariner-addon/pull/500

repo: stolostron/submariner-addon


text:
proxy/mock mixed behaviour

User story.
As a backend developer I want to let consumers test against my API Mocks and at the same time record information about structural mismatches with the backend answer.
Is your feature request related to a problem?
It's a workflow improvement to continuously record where structural mismatches exists while the real API is being continuously build.
Describe the solution you'd like
I'd like to run the PRISM in proxy mode pointing to the real API and generating the violation hints. At the same time I'd like PRISM to answer with mock data which would have matched as if it runs in mock mode. It's a proxy/mock mixed behaviour.

Hey,
your use case looks a little bit weird to me. We do not have support for this currently in Prism and I  do not think it's coming anytime soon.
Can you clarify a little bit the specific use case?

@XVincentX this was a feature of prism in Stoplight Classic. You could turn mocking on/off for particular operations in your API spec with extensions, you could also use a before script to control the behavior as well.

Sorry I’m not sure I understand the use case.
If you’re running a mock server, mismatched requests will show up in sl-violations, the same as if you ran the proxy.
The proxy is when you want to talk to a real server. The mock is for when you don’t want to talk to a real server.
Could you describe the use case where you need both?

The basic idea is finding (edge) cases where responses don't match the openapi spec.
It seemed easy to do with the features in prism. But I now realize this should be better done on its own with a tool like https://dredd.org
Sorry to have bothered you two :/

There's a bunch of options: https://apisyouwonthate.com/blog/keeping-documentation-honest
Dredd is not maintained much these days, OpenAPI v3 support only ever hit experimental, and it can be a bit of a pain to set up an effective seed anyway. If you go with it, enable the --sorted option to do all your GETs first and DELETEs at the end - that'll help a lot.
If you're concerned about keeping design and code in sync there's other ways of getting that sorted which don't involve a special test tool. More on this here: https://www.apisyouwonthate.com/blog/api-design-first-vs-code-first/
Got another article coming on that blog which drives the point home even harder but it's 24 hours from being ready. Check back tomorrow :D

Not sure if it's the use case @hottehead suggested, but if i have several (let's say 4) Endpoints, I'd be interested in implementing one, which I can then switch from mock to proxy (for example by using an extenstion in the spec, like x-proxy) while the other 3 Endpoints are still mocked.

Yeah this is something that Prism 2 was offering — we can probably enable it back sometime in the future.

authors: WoH XVincentX hottehead philsturgeon pytlesk4

repo: stoplightio/prism


text:
Verbose logging

User story.
As a Prism CLI user, I can supply --verbose (or --debug) CLI option, so that I can see more verbose logs.
Is your feature request related to a problem?
There are occasions when I do not have easy access to request or response payload. For those cases, it would be helpful if I could see the actual request and response payloads together with headers. That should help me debug possible problems with spec or request data.
Describe the solution you'd like
A -d|--debug or --verbose CLI option which enables the above functionality.
Additional context
This could be also super helpful in debugging callbacks, which are performing asynchronous request based on previous req/res pair.

Aha! I can understand the frustration while debugging the thing.
I think we have two steps here to do:

Logging the things that we need
Forward the filtering capabilities to Pino

Right now we have already several log levels that come bundled with pino

trace, debug, info, warn, error, and fatal.

On top of these, we add some other that are used just for beautifying purposes:

note, success, start

I've prepared #780 to clean up some stuff that effectively does not make much sense anymore and inject the custom logging logic in the CLI instead of keeping them in the main package.
Once such PR is merged we can log the stuff you think it's important, review the levels and then the filtering capability.

In progress https://github.com/stoplightio/prism/pull/801

Hey all, wanted to check to see if this feature is still in progress? It looks like it had some momentum during the winter but fell off. Is this still planned? Would love to be able to use this.

Yeah we have paused working on this for two reasons:

We are busy on doing something else and this feature is not exactly that critical
We are generally rethinking a little bit the way we do logging and it is kind of probably that we are going to change the way we output messages (specifically speaking, changing the abstraction from ReaderTaskEither to a Writer<TaskEither>> — but we still need to check out if it's worth doing and makes sense for real.

Logging correctly is hard — that's for sure.

@XVincentX how's this one come along? Is it going to happen or should we close it down?

I do not think this is gonna happen anytime soon, but the content is still valid.

@ryotrellim hey would you like to close this or prioritise it? It’s been around a while but could be helpful.

@XVincentX and other maintainers – I'm interested in having someone on my team put up a PR for this early next week. Any tips/pointers/thoughts on how to get started or what you would like to see from it?

I no longer work at Stoplight. You might want to mention somebody else to get attention!

@XVincentX hey fella, thanks for pinging this onto my inbox! 🙌🏻
@rattrayalex could you hop onto the community discord where you can chat with some of the other Prism developers?
https://discord.gg/stoplight
If you don’t use discord no worries, I can share this with some of the team.

authors: TylerNielsen XVincentX karol-maciaszek philsturgeon rattrayalex


text:
v4.3.2

Releases version 4.3.2 of prism

Fixed @rainum


text:
Adds a check for errors on useAsyncStoryblok to prevent errors from gettting to clients.

This change adds a check agains the error object coming from useAsyncData to prevent axios errors getting down to the client.
This is how it looks without the check:

With the check:

This changes has been tested against my content project.
Note:
I am not sure on the method to check if a proper result is enough or if you have a more preferred way of doing it.

I'm closing this PR, so we can merge the solution implemented with createError being thrown instead https://github.com/storyblok/storyblok-nuxt/pull/295

authors: Dawntraoz SebbeJohansson

repo: storyblok/storyblok-nuxt


text:
[Improve] Upgrade to storybook 7

Upgrade to storybook 7
Current version doesnt work with storybook 7

`Hi @AgainstLightning  here is a repo with most custom render function you use, no need any addon
https://github.com/chakAs3/vue3-storybook-custom-render

there is with other implementations
https://github.com/chakAs3/vue3-pinia-storybook
https://github.com/storybookjs/vue3-code-examples

Note: we used this addon successfully with SB7, but it finally broke for us as well with SB8.
I built a new addon that does the same thing, basically from scratch: https://github.com/mihkeleidast/storybook-addon-source

authors: chakAs3 mihkeleidast volnei

repo: storybookjs/addon-jsx


text:
preset-create-react-app support of craco

When @craco/craco is used as a replacement of react-scripts, there is no fork of react-scripts, thus this line, which tries to require the non-existing module .../@craco/craco/index.js/config/webpack.config, will fail.
Although we could solve the issue by either providing a local react-scripts or mocking the module, I  want to know whether there is any plan for preset-create-react-app to support user provided entire webpack config. Thanks a lot.
The storybook config looks like
addons: [
    {
        name: "@storybook/preset-create-react-app",
        options: {
            scriptsPackageName: "@craco/craco"
        }
    },
    ...
]

Hi @liokm, so as craco is a little different than a fork of react-scripts, it requires you to modify the CRA config for Storybook in the same way that you do with Craco.
Can you share more information about your Craco setup? It might be something we can automate.

Hi @mrmckeb , thank you.
The config is as following. I am not using preset-ant-design as there are more configures to be done. antd.less could be changed to modifyVars of lessOptions and does not matter.
Yeah, craco reads CRA webpack config and does its injection before invoking react-scripts commands.
[root]/craco.config.js
const CracoAntDesignPlugin = require("craco-antd");
const path = require("path");

module.exports = {
  plugins: [
    {
      plugin: CracoAntDesignPlugin,
      options: {
        customizeThemeLessPath: path.join(__dirname, "src/antd.less")
      }
    }
  ]
};

package.json
    "@craco/craco": "^5.6.3",
    "@storybook/preset-create-react-app": "^1.5.2",
    "@storybook/react": "^5.3.14",
    "antd": "^4.0.1",
    "craco-antd": "^1.14.1",

Hi @liokm, from what I can see, you'll need to implement this yourself for now. As Craco isn't a fork of CRA, but instead modifies it, you would need to do something like this:
addons: [
    '@storybook/preset-create-react-app'
],
webpackFinal: async (config) => {
    // Change your config here

    return config;
  },

You can find more about webpackFinal here: https://storybook.js.org/docs/configurations/custom-webpack-config/#examples
From what I can see, you shouldn't need to change too much for Ant Design, but it looks like that Craco plugin does a lot:
https://github.com/DocSpring/craco-antd/blob/master/lib/craco-antd.js
We'd welcome a PR to add some level of compat with Craco, but right now it seems that this would be a large amount of work. CC @shilman who may have some other thoughts?

I just moved to using craco to apply the craco-cesium fixes to use cesium.
I was able to get it working for that specific craco plugin with the following main.js:

module.exports = {
    stories: ["../src/**/*.stories.tsx"],
    addons: [
        "@storybook/preset-create-react-app",
        "@storybook/addon-actions",
        "@storybook/addon-links",
    ],
    webpackFinal( config, { configType } ) {
        // run craco plugins since there is no preset for craco
        return cracoCesium.overrideWebpackConfig( {
            webpackConfig: config,
            context: {
                env: configType.toLowerCase(),
            },
        } );
    },
};

I hit problems with Craco and storybook last night so built an experimental preset based on preset-create-react-app to use the webkit config from Craco instead of pulling it directly from react-scripts.
This means any Craco plugins, changes postcss plugins, manual webkit tweaks, etc will also be applied to storybook.
Its a bit hackey at the moment and calls out to internal apis inside preset-create-react-app but that seemed like a better choice than re-inventing the wheel.
Any feedback, contributions, fixes, etc are welcome, it would be good to have solid integration between storybook and Craco.

Github Repo
Yarn
NPM

@danielknell first, thanks! Worked a treat and seems like the most sensible+DRY way of handling craco configs.
FYI: I'm using craco to customise babel-loader so that I can have TSX outside my CRA root (yarn workspace monorepo setup). Your preset worked almost out of the box for me, except that I had to set the cracoConfigFile option because my craco config lives in my workspace root, not in the CRA root. That could be made a little bit smoother if the preset checked package.json for the cracoConfig key. I'll put in a PR if I get chance, but just letting you know. Cheers again!

@danielknell first, thanks! Worked a treat and seems like the most sensible+DRY way of handling craco configs.
FYI: I'm using craco to customise babel-loader so that I can have TSX outside my CRA root (yarn workspace monorepo setup). Your preset worked almost out of the box for me, except that I had to set the cracoConfigFile option because my craco config lives in my workspace root, not in the CRA root. That could be made a little bit smoother if the preset checked package.json for the cracoConfig key. I'll put in a PR if I get chance, but just letting you know. Cheers again!

Thanks a lot @danielknell for your work on https://github.com/artisanofcode/storybook-preset-craco cause it works great!

repo: storybookjs/presets


text:
Documentation for using with Cypress

I was able to use this successfully with Jest, but I have been unable to find a way to get this working with Cypress component testing.
There's an article describing this working, https://www.cypress.io/blog/2021/04/06/cypress-component-testing-react/
The examples given don't describe how to fully set this up, as it looks like there is setup in cypress/plugins that needs to be done for this to work properly, but it requires react-scripts/cra. Can we get documentation of how to use this with Storybook alone?

Hey there! I wanted to thank you for using @storybook/testing-react!
@storybook/testing-react has been promoted to a first-class Storybook functionality in Storybook 7. This means that you no longer need this package, and this package will not be worked on anymore (especially regarding Storybook 6, unless there are security issues). Instead, you can import the same utilities, but from the @storybook/react package. Additionally, the internals of composeStories and composeStory have been revamped, so the way a story is composed is way more accurate, and it's possible this issue doesn't happen there anymore.
Please do the following:

Upgrade to Storybook 7 if you haven't already
Uninstall @storybook/testing-react
Update your imports from @storybook/testing-react to @storybook/react

// Component.test.jsx
- import { composeStories } from '@storybook/testing-react';
+ import { composeStories } from '@storybook/react';

// setup-files.js
- import { setProjectAnnotations } from '@storybook/testing-react';
+ import { setProjectAnnotations } from '@storybook/react';

For now, I'll be closing this issue. Please, if even after migrating, you are still experiencing issues, report them in the Storybook monorepo.
Thank you so much for this journey!

authors: sdaconceicao yannbf

repo: storybookjs/testing-react


text:
fix: support addons that use hook api (#30)

Issue: #30
What Changed
Fixed the following error when using the hooks function (ex. useArg) in the decorator function.
      Error: Uncaught [Error: Storybook preview hooks can only be called inside decorators and story functions.]

Checklist
Check the ones applicable to your change:

[X] Tests are updated
[X] Documentation is updated

Change Type
Indicate the type of change your pull request is:

[ ] documentation
[X] patch
[ ] minor
[ ] major

Hey @uk-taniyama thank you so much for your contribution! Looks great 🚀

authors: uk-taniyama yannbf


text:
Loop Auto Logout Error

I get a message that I'm logged in, but I'm still logged out.

Check if the login task has been rejected.
const login = client.login(...);
login.then(() => console.log('Logined Successfully');
login.catch((e) => console.error(e));

Check if the login task has been rejected.
const login = client.login(id, pass, uuid);
login.then(() => console.log('Logined Successfully');
login.catch((e) => console.error(e));

Result is equal

Cannot reproduce bug.

locoManager: LocoManager {
  locoSocket: [LocoSecureSocket],
  expireTime: 600,
  handler: [TalkPacketHandler],
  locoConnected: false,
  locoLogon: false
}

Logout Try -> Error: Not logon at TalkClient.logout

@storycraft plz fix error

authors: Mage-Megumin delta-kor storycraft

repo: storycraft/node-kakao


text:
[Suggestion] Allow Scene's children property to work with Arrays

Something like this:
sprites = [player, enemy1, enemy2, enemy3, ...]
scene = Scene({
	id: 'game',
	children: [tileEngine, sprites],
	cullObjects: false
})

You can use spread operator or concat method
console.log([tileEngine, ...sprites])
console.log([tileEngine].concat(sprites))
console.log(sprites.concat(tileEngine, anotherSprite, ...))

Thanks! i'm learning Javascript but i really don't know why i didn't think that.

repo: straker/kontra


text:
Be able to choose which content types that url alias applies to

Feature request
Summary
For some content types, it does not make sense to have a url. But for all content types there are shown a url alias field.
Why is it needed?
It can be confusing for the end user why there is a url field for content types that does not have a url.
Suggested solution(s)

Make it possible to define which content types url alias applies to (or should be ignored) in the plugins config file

Duplicate of #15

Thanks!

authors: Alexnortung boazpoolman


text:
Fixed bind mount paths

I right now ran into this and didn't get why the container didn't write back the changes to the host files.
Turns out the bind mount path were wrong and were pointing into /opt/app instead of /app.

I think this might be wrong @nevotheless
Thank you again for opening a PR.
https://github.com/strapi-community/strapi-tool-dockerize/blob/main/templates/Dockerfile.liquid#L26
We are looking inside for /opt/app not /app

That's weird, the generated Dockerfile i got looks like this

FROM node:16-alpine as build
# Installing libvips-dev for sharp Compatability
RUN apk update && apk add build-base gcc autoconf automake zlib-dev libpng-dev vips-dev && rm -rf /var/cache/apk/* > /dev/null 2>&1
ARG NODE_ENV=production
ENV NODE_ENV=${NODE_ENV}
WORKDIR /opt/
COPY ./package.json ./yarn.lock ./
ENV PATH /opt/node_modules/.bin:$PATH
RUN yarn config set network-timeout 600000 -g && yarn install
WORKDIR /opt/app
COPY ./ .
RUN yarn build

FROM node:16-alpine
RUN apk add vips-dev
RUN rm -rf /var/cache/apk/*
ARG NODE_ENV=production
ENV NODE_ENV=${NODE_ENV}
WORKDIR /app
COPY --from=build /opt/node_modules ./node_modules
ENV PATH /opt/node_modules/.bin:$PATH
COPY --from=build /opt/app ./
EXPOSE 1337
CMD ["yarn", "develop"] // i changed this to develop for the time being

Yeah to verify i just ran
npx @strapi-community/dockerize new --dbclient=mysql --dbhost=localhost --dbport=1234 --dbname=strapi --dbusername=strapi --dbpassword=strapi --projecttype=js --packagemanager=yarn --usecompose=true --env=both

in an empty directory the outcome was this Dockerfile

But WORKDIR is only set in the builder stage not in the actual "final" stage. When i get into the container the immediate directory is /app like it is stated in the final stage.
https://github.com/strapi-community/strapi-tool-dockerize/blob/main/templates/Dockerfile-prod.liquid this file seems to be used for the command hence the /app directory i assume.

I'll check it out then because then we rather want to fix the final stage.
Else it would break the rest.
If you care to check or fix the final stage then I bee happy to merge it.

Yeah i can check it out and fix it. Just need to find out how to use the generator shenanigans locally without using npx @strapi-community/dockerize.

out how to us

Clone down the repo.
then do npm link in it and you can use strapi-dockerize to test it locally :)
Thank you again.

authors: Eventyret nevotheless

repo: strapi-community/strapi-tool-dockerize


text:
[bug]: SVG theming API bug

What version of strapi-design-system are you using?
Bug report
All the icons inside the BaseButton on https://design-system-git-main-strapijs.vercel.app/?path=/story/design-system-technical-components-basebutton--base have their colors resetted by an internal CSS rule.
Required System information

Npm version: Not applicable
NodeJS version: Not applicable
Strapi Design System version: 1.4.0
Browser : Not applicable

What's Wrong?
All the icons inside the BaseButton on https://design-system.strapi.io/colors have their colors resetted by an internal CSS rule and then in the Strapi CMS to revert back their colors we need to overwrite the CSS
To Reproduce
We discovered the issue when we investigated this bug, https://github.com/strapi/design-system/issues/793, you can reproduce the issue simply using for example a LinkButton with an icon https://design-system-git-main-strapijs.vercel.app/?path=/story/design-system-components-linkbutton--icons
Expected Behaviour
It overwrites the svg path color with the value neutral0 for the theme (light #ffffff dark #212134)

Actually, this is more a bad api restriction than a bug. This was done voluntarily because it was not planned to have a different icon color for this component.
If we open the api of this component, we have to make sure that the buttons that use BaseButton are not broken and that the colors of the icons follow the colors of the variants. https://github.com/strapi/design-system/blob/main/packages/strapi-design-system/src/Button/utils.js#L102

solved by #1696

authors: HichamELBSI joshuaellis simotae14

repo: strapi/design-system


text:
feat: MPTT relations compatibility issue reproduction

Description
Trying to get an author and then the total count of the children of that author breaks when there are no children present but requested in the query.
Error:
"message": "An unknown error occurred."

The PR includes a test test_nexted_children.py, which illustrates the use case and resulting error.
FYI: Disabling the DjangoOptimizerExtension makes the test pass without any issue.
Types of Changes

[x] Bug reproduction for 3rd party compatibility

Summary by Sourcery
This pull request enhances the RelayAuthor model by adding MPTT relations to support hierarchical data structures. Additionally, a new test case is introduced to verify the nested children count functionality.

Enhancements:

Added MPTT (Modified Preorder Tree Traversal) relation to the RelayAuthor model to support hierarchical data structures.

Tests:

Introduced a new test case 'test_nested_children_total_count' to verify the nested children count functionality in the RelayAuthor model.

Closing this as https://github.com/strawberry-graphql/strawberry-django/pull/553 superseded it

repo: strawberry-graphql/strawberry-django


text:
Getting "got an unexpected keyword argument 'type_'" with basic example

I've been banging my head against this for a little while and compared to the examples on here, but haven't been able to figure this one out.
My model
from django.db import models

class Contact(models.Model):
    first_name = models.CharField(max_length=100)
    last_name = models.CharField(max_length=100)
    email = models.EmailField()

Strawberry types
import strawberry
from strawberry.django import auto
from core import models

@strawberry.django.type(models.Contact)
class Contact:
    id: auto

Error
  File "Projects/project/project/urls.py", line 19, in <module>
    from core.schema import schema
  File "Projects/project/core/schema.py", line 3, in <module>
    from core.types import Contact
  File "Projects/project/core/types.py", line 7, in <module>
    class Contact:
  File "Projects/project/env/lib/python3.9/site-packages/strawberry_django/type.py", line 154, in wrapper
    return process_type(cls, model, filters=filters, **kwargs)
  File "Projects/project/env/lib/python3.9/site-packages/strawberry_django/type.py", line 131, in process_type
    fields = get_fields(django_type)
  File "Projects/project/env/lib/python3.9/site-packages/strawberry_django/type.py", line 89, in get_fields
    field = get_field(django_type, field_name, field_annotation)
  File "Projects/project/env/lib/python3.9/site-packages/strawberry_django/type.py", line 29, in get_field
    field = StrawberryDjangoField(
  File "Projects/project/env/lib/python3.9/site-packages/strawberry_django/fields/field.py", line 60, in __init__
    super().__init__(graphql_name=graphql_name, python_name=python_name, **kwargs)
  File "Projects/project/env/lib/python3.9/site-packages/strawberry_django/ordering.py", line 56, in __init__
    super().__init__(**kwargs)
  File "Projects/project/env/lib/python3.9/site-packages/strawberry_django/filters.py", line 138, in __init__
    super().__init__(**kwargs)
  File "Projects/project/env/lib/python3.9/site-packages/strawberry_django/pagination.py", line 24, in __init__
    super().__init__(**kwargs)
TypeError: __init__() got an unexpected keyword argument 'type_'

Any ideas where I'm going wrong, or is this a bug?
Thanks

@lohmander can show me what version of strawberry are you using?
we just release a big internal update and it might have broken strawberry django

Sure thing:
strawberry-graphql==0.69.2
strawberry-graphql-django==0.2.3

@lohmander can you try to install strawberry-graphql<0.69 ?
we'll ship a fix for the latest version soon 😊

@patrick91 this did indeed fix the issue 👍 thanks!

Temporal solution #47 and relesed. Please update to v0.2.4. Proper fix (#48) will be done later.


text:
Streama fails to use MySQL Database

Issue description
I am trying to test run Streama using MySQL Database as back end. I have correctly (I believe) setup application.yml to use a different port and MySQL as data source. While the port change takes effect, Streama however still uses the the inbuil H2 database instead of MySQL
Steps to Reproduce
Create Streama database on MySQL Server backend
Configure application.yml (actual content included in ENVIRONMENT INFORMATION below)
Run Streama (java -jar streama-1.6.0-FINAL.war)
Expected Behaviour
Streama is expected to connect to MySQL , create requisite tables and use Database specified in application.yml
Actual Behaviour
Streama uses in built H2 Database (evidenced by creation of streama.mv.db & streama.trace.db). No tables are created in the MySQL Database specified in application.yml
Environment Information
Operating System: Ubuntu Server 18.10 x64 Server running MySQL Server 5.7.24
Streama version: v1.6.0 FINAL
Custom streama build, describe customizations or provide link to fork (If Applicable):
application.yml Content
environments:
production:
dataSource:
        #driverClassName: 'com.mysql.jdbc.Driver'
        driverClassName: 'com.mysql.cj.jdbc.Driver'

        #url: jdbc:mysql://localhost/streama
        url: jdbc:mysql://localhost/streama?useLegacyDatetimeCode=false&serverTimezone=UTC&useSSL=false

        username: admin
        password: admin

server:
port: 6789
Container Version (If Applicable): /

Try to use this
environments:      production:         dataSource:             dbCreate: update             driverClassName:  'com.mysql.cj.jdbc.Driver'             url: jdbc:mysql://localhost:----insert-port---/streamadb?useLegacyDatetimeCode=false&serverTimezone=UTC&useSSL=false             username: streamausername             password: streampass

Hi @EchoDev thanks for your response. Got around to trying this both on the 1.6.0 and 1.6.1. Doesn't work. Streama still doesn't use MySQL even though it runs on the port specified in application.yml. I tried both http://localhost:mysqlPort and http://server-ip:mysqlPort.

Can you tell us how your set up looks like? In what folder do you have the application.yml, what commands are you using, what is the console output etc. I can compare to what it looks like on my end.

Streama 1.6.1.war and application.yml are both in /root.
I'm running streama by issuing the following command
java -jar streama-1.6.1.war

can you paste the contents of your entire application.yml pls?

environments: production: dataSource: dbCreate: update driverClassName: 'com.mysql.cj.jdbc.Driver' url: jdbc:mysql://localhost:----insert-port---/streamadb?useLegacyDatetimeCode=false&serverTimezone=UTC&useSSL=false username: mysqlusername password: mysqluserpass

Bump

authors: EchoDev airsay dularion

repo: streamaserver/streama


text:
Fix useLocalStorage types for functional updates

Fixes #438

Any interest in this PR?

Great addition, thanks 👍

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

authors: charlax streamich wardoost

repo: streamich/react-use

license: Unlicense


text:
readdir fails with error when one of the fs fails

The readdir method is not working properly since it fails if one of the file systems registered on the union file system fails.
Environment:

Node: v10.16.3
unionfs: 0b5f0368af01679b31c0ed1017716bf396c63440

Sample:
This is a sample of what I'm trying to do:
const realFs = require('fs');
const virtualFs = new (require('memfs').Volume)();
const unionFs = new (require('unionfs').Union)();

virtualFs.fromJSON({
	'does-not-exist-in-real-fs/file.txt': ''
});

unionFs.use(virtualFs);
unionFs.use(realFs);

Current behavior:
If readdir has an error in one of the file systems registered on unionfs, it will have an error also in unionfs.
virtualFs.readdir('does-not-exist-in-real-fs', callback)); // called without error
unionFs.readdir('does-not-exist-in-real-fs', callback)); // called with error
realFs.readdir('does-not-exist-in-real-fs', callback)); // called with error

Expected result:
The directory can be read in the virtual file system, union file system but not the real file system.
virtualFs.readdir('does-not-exist-in-real-fs', callback)); // called without error
unionFs.readdir('does-not-exist-in-real-fs', callback)); // called without error
realFs.readdir('does-not-exist-in-real-fs', callback)); // called with error

:tada: This issue has been resolved in version 4.2.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

authors: magarcia streamich

repo: streamich/unionfs


text:
Fix scroll padding, so headers don't appear behind the top bar when you navigate to a page via an #anchor.

📚 Context
When you click navigate to a page like https://docs.streamlit.io/develop/api-reference/utilities/st.context#contextcookies, which has a hash pointing to a specific part of the page (#contextcookies in this case), that part of the page gets partially obscured by the site's top bar. With this change, this no longer happens.
Compare the above with https://deploy-preview-1132--streamlit-docs.netlify.app/develop/api-reference/utilities/st.context#contextcookies! Here you'll the "context cookies" header is visible rather than hidden under the top bar.
🧠 Description of Changes
Fix the number used in scroll padding to match what we use for the top bar height.
💥 Impact
Size:

[x] Small
[ ] Not small

🌐 References
None

Contribution License Agreement
By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

@sfc-gh-tteixeira Thanks! This is another nit I can check off my list. Just FYI, this PR ends up adding excessive padding in other areas. I think it's still better than what it was since at least no header is hidden now, but we might be able to target the scroll padding a little more. See preview vs current.

authors: sfc-gh-dmatthews sfc-gh-tteixeira

repo: streamlit/docs


text:
[Snyk] Fix for 1 vulnerabilities

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

pom.xml

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

688/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 5.9
Denial of Service (DoS) SNYK-JAVA-COMFASTERXMLJACKSONCORE-3038424
com.fasterxml.jackson.core:jackson-databind:  2.13.2.1 -> 2.13.4  com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:  2.13.2 -> 2.13.4
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

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
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)

 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.

authors: BewareMyPower CLAassistant

repo: streamnative/kop


text:
From our "private" package (it is never published) we should point to ...

a) a file that actually exists
b) it should be the private wrapper API so that we use this conveniently for dev/testing modes
c) this fix is also needed to make this package work in context of npm link

:+1:

authors: maxbok tchakabam


text:
Record as string

How can I get a row of CSV files as a String rather than a struct or record? for example
My file is :
a,b,c
d,e,f
I want the records to look like
a,b,c
d,e,f
and not like
{schema:{"type":"struct","fields":[{"type":"string","field":"message"}]}, payload:{message:"a,b,c"}}
{schema:{"type":"struct","fields":[{"type":"string","field":"message"}]}, payload:{message:"d,e,f"}}

Hey @pradeepsh2203
try to set value.converter.schemas.enable: false

repo: streamthoughts/kafka-connect-file-pulse